Re: PATCH: pgagent connection string parsing - Mailing list pgadmin-hackers

From Dave Page
Subject Re: PATCH: pgagent connection string parsing
Date
Msg-id CA+OCxozt7KbcqNvnd43rKn6QWKSYiaMUaqtvm1skaJefo=AfTw@mail.gmail.com
Whole thread Raw
In response to Re: PATCH: pgagent connection string parsing  (Ashesh Vashi <ashesh.vashi@enterprisedb.com>)
Responses Re: PATCH: pgagent connection string parsing  (Thomas Krennwallner <tk+pgsql@postsubmeta.net>)
List pgadmin-hackers


On Mon, Aug 21, 2017 at 9:46 AM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
On Mon, Aug 21, 2017 at 1:41 PM, Dave Page <dpage@pgadmin.org> wrote:


On Mon, Aug 21, 2017 at 6:24 AM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:


On Aug 21, 2017 10:52, "Thomas Krennwallner" <tk+pgsql@postsubmeta.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@postsubmeta.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

pgadmin-hackers by date:

Previous
From: Ashesh Vashi
Date:
Subject: Re: PATCH: pgagent connection string parsing
Next
From: Teng Zhang
Date:
Subject: Re: [pgadmin4][Patch] Greenplum specific DDL and Dashboard display