| From: | Alex <ash(at)commandprompt(dot)com> |
|---|---|
| To: | Marko Kreen <markokr(at)gmail(dot)com> |
| Cc: | Daniel Farina <daniel(at)heroku(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Another review of URI for libpq, v7 submission |
| Date: | 2012-03-20 22:18:44 |
| Message-ID: | 8762dyuda3.fsf@commandprompt.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Marko Kreen <markokr(at)gmail(dot)com> writes:
> On Thu, Mar 15, 2012 at 11:29:31PM +0200, Alex wrote:
>> https://github.com/a1exsh/postgres/commits/uri
>
> The point of the patch is to have one string with all connection options,
> in standard format, yes? So why does not this work:
>
> db = PQconnectdb("postgres://localhost");
>
> ?
Good catch.
I've figured out that we'll need a bit more intrusive change than simply
overriding the expand_dbname check in conninfo_array_parse (like the
current version does) to support URIs in all PQconnect* variants.
I still need to figure out some details, but this is to give people a
status update.
--
Alex
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2012-03-20 23:05:58 | pgsql: Improve the -l (limit) option recently added to contrib/vacuumlo |
| Previous Message | Greg Stark | 2012-03-20 22:16:55 | Re: Memory usage during sorting |