Re: PATCH: pgagent connection string parsing

On Mon, Aug 21, 2017 at 9:46 AM, Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com
> wrote:

> On Mon, Aug 21, 2017 at 1:41 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>>
>>
>> On Mon, Aug 21, 2017 at 6:24 AM, Ashesh Vashi <
>> ashesh(dot)vashi(at)enterprisedb(dot)com> wrote:
>>
>>>
>>>
>>> On Aug 21, 2017 10:52, "Thomas Krennwallner" <tk+pgsql(at)postsubmeta(dot)net>
>>> wrote:
>>>
>>> On Mon Aug 21, 2017 06:50:03AM +0530, Ashesh Vashi wrote:
>>> > On Aug 20, 2017 22:48, "Thomas Krennwallner" <[1]
>>> tk+pgsql(at)postsubmeta(dot)net>
>>> > wrote:
>>> >
>>> > % ./pgagent -f -t60 -l2 dbname=xxx user=tkren
>>> connection_timeout=5
>>> [...]
>>> > Sun Aug 20 18:24:34 2017 : DEBUG: Creating DB connection:
>>> user=tkren connection_timeout=5 dbname=xxx
>>> > Sun Aug 20 18:24:34 2017 : WARNING: Couldn't create the primary
>>> connection (attempt 1): invalid connection option "connection_timeout"
>>> >
>>> > This suggests pgAgent is not using the latest version of libpq.
>>> > Please use LD_LIBRARY_PATH environment to inform pgAgent to use the
>>> libpq,
>>> > it is compiled with.
>>> > -- Thanks,
>>> > Ashesh Vashi
>>> > -- Thanks, Ashesh
>>>
>>> Unfortunately, this does not work. On a current Debian sid system,
>>> I've compiled pgagent from source and get
>>>
>>> % ldd ./pgagent
>>> [...]
>>> libpq.so.5 => /usr/lib/x86_64-linux-gnu/libpq.so.5
>>> (0x00007ff531382000)
>>> [...]
>>>
>>>
>>> The system has libpq 9.6.4 installed:
>>>
>>> % apt-cache show libpq5
>>> Package: libpq5
>>> Source: postgresql-9.6
>>> Version: 9.6.4-1
>>> [...]
>>>
>>> You may want set the rpath manually using chrpath utility for testing.
>>>
>>>
>> Regardless of that, connection.cpp does need to be taught the correct
>> name for connect_timeout, as well as the additional appname related
>> parameters right?
>>
> True.
>

Thanks for volunteering :-)

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company