Re: [PATCH] Add PQconninfoParseParams and PQconninfodefaultsMerge to libpq

Поиск
Список
Период
Сортировка
От Phil Sorber
Тема Re: [PATCH] Add PQconninfoParseParams and PQconninfodefaultsMerge to libpq
Дата
Msg-id CADAkt-jwS=rK0+yCmtQPosAsRCXCyyWwMjZUw2iBU3ghFGvPOg@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [PATCH] Add PQconninfoParseParams and PQconninfodefaultsMerge to libpq  (Alvaro Herrera <alvherre@2ndquadrant.com>)
Ответы Re: [PATCH] Add PQconninfoParseParams and PQconninfodefaultsMerge to libpq  (Alvaro Herrera <alvherre@2ndquadrant.com>)
Список pgsql-hackers
On Mon, Feb 4, 2013 at 10:16 AM, Alvaro Herrera
<alvherre@2ndquadrant.com> wrote:
> Phil Sorber wrote:
>> On Mon, Feb 4, 2013 at 9:13 AM, Alvaro Herrera <alvherre@2ndquadrant.com> wrote:
>
>> > Uh, no existing code can use this new functionality?  That seems
>> > disappointing.
>>
>> I wrote this because I wanted to use it in pg_isready. I also wrote
>> something for pg_isready to get around not having this. I think adding
>> these two functions to libpq would be the better option, but wanted
>> something that could fix an existing issue without having to patch
>> libpq so late in the 9.3 development process. Actually, I think it was
>> you who suggested that approach.
>
> Yes, I realize that (and yes, I did).  But is no code other than
> pg_isready doing this?  Not even the libpq URI test program?

I think it probably would be able to benefit from this. Are you
suggesting I patch that too? I thought it was usually frowned upon to
touch random bits of working code like that. I'd be more than happy to
do it if it helps build the case for getting this added.

>
> --
> Álvaro Herrera                http://www.2ndQuadrant.com/
> PostgreSQL Development, 24x7 Support, Training & Services



В списке pgsql-hackers по дате отправления:

Предыдущее
От: Alvaro Herrera
Дата:
Сообщение: Re: [PATCH] Add PQconninfoParseParams and PQconninfodefaultsMerge to libpq
Следующее
От: Alvaro Herrera
Дата:
Сообщение: Re: [PATCH] Add PQconninfoParseParams and PQconninfodefaultsMerge to libpq