From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> |
Cc: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
Subject: | Re: Improving isolationtester's data output |
Date: | 2021-06-15 23:26:25 |
Message-ID: | 583674.1623799585@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> writes:
> On 2021-Jun-15, Tom Lane wrote:
>> If we wanted to buy into that, I'd think about discarding this
>> ad-hoc code altogether in favor of using one of libpq's fe-print.c
>> routines. But I'm not really sure that the small legibility gains
>> that would result are worth massive changes in the output files.
> Shrug -- it's a one time change. It wouldn't bother me, for one.
Going forward it wouldn't be a problem, but back-patching isolation
test cases might find it annoying. On the other hand, my nearby
patch to improve isolation test stability is already going to create
issues of that sort. (Unless, dare I say it, we back-patch that.)
I do find it a bit attractive to create some regression-testing
coverage of fe-print.c. We are never going to remove that code,
AFAICS, so getting some benefit from it would be nice.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Jacob Champion | 2021-06-15 23:50:14 | Re: Support for NSS as a libpq TLS backend |
Previous Message | Alvaro Herrera | 2021-06-15 23:20:11 | Re: Improving isolationtester's data output |