| From: | Michael Paquier <michael(at)paquier(dot)xyz> |
|---|---|
| To: | Jacob Champion <pchampion(at)vmware(dot)com> |
| Cc: | "peter(dot)eisentraut(at)enterprisedb(dot)com" <peter(dot)eisentraut(at)enterprisedb(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Proposal: Save user's original authenticated identity for logging |
| Date: | 2021-09-23 10:34:36 |
| Message-ID: | YUxYPBLOLrXWulDh@paquier.xyz |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Wed, Sep 22, 2021 at 03:18:43PM +0000, Jacob Champion wrote:
> On Wed, 2021-09-22 at 10:20 +0200, Peter Eisentraut wrote:
>> This should be added to each level of a function call that represents a
>> test. This ensures that when a test fails, the line number points to
>> the top-level location of the test_role() call. Otherwise it would
>> point to the connect_ok() call inside test_role().
>
> Patch LGTM, sorry about that. Thanks!
For the places of the patch, that seems fine then. Thanks!
Do we need to care about that in other places? We have tests in
src/bin/ using subroutines that call things from PostgresNode.pm or
TestLib.pm, like pg_checksums, pg_ctl or pg_verifybackup, just to name
three.
--
Michael
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Ranier Vilela | 2021-09-23 11:17:31 | Re: Eval expression R/O once time (src/backend/executor/execExpr.c) |
| Previous Message | Peter Eisentraut | 2021-09-23 10:27:17 | Re: logical decoding and replication of sequences |