From: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> |
---|---|
To: | Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com>, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> |
Cc: | Ashutosh Bapat <ashutosh(dot)bapat(at)enterprisedb(dot)com>, Christoph Berg <myon(at)debian(dot)org>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Log LDAP "diagnostic messages"? |
Date: | 2017-09-19 19:57:45 |
Message-ID: | 09ad0092-f026-3957-4599-c57ad1dca79c@2ndquadrant.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 9/15/17 08:43, Thomas Munro wrote:
> On Fri, Sep 15, 2017 at 2:12 AM, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> wrote:
>> I think the ldap_unbind() changes should be in a separate preliminary
>> patch to be committed separately and backpatched.
>
> OK, here it is split into two patches.
I've looked this over.
In the 0001 patch, I would move the ldap_unbind() calls after the
ereport(LOG) calls. We do all the other resource cleanup (pfree() etc.)
after the ereport() calls, so it would be weird to do this one
differently. Also, in the second patch you move one of the
ldap_unbind() calls down anyway.
In the 0002 patch, I think this is a bit repetitive and could be
refactored even more. The end result could look like
ereport(LOG,
(errmsg("blah"),
errdetail_for_ldap(ldap)));
ldap_unbind(ldap);
--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Alexander Korotkov | 2017-09-19 20:03:04 | Re: Re: proposal - psql: possibility to specify sort for describe commands, when size is printed |
Previous Message | Andrew Dunstan | 2017-09-19 19:51:39 | Re: Show backtrace when tap tests fail |