Horton

Thread

Commits

  1. Reorder connection markers in loadbalance tests

  2. Support connection load balancing in libpq

  1. Reorder connection markers in libpq TAP tests

    Gurjeet Singh <gurjeet@singh.im> — 2023-04-21T06:00:00Z

    Commit 7f5b198 introduced TAP tests that use string literals to mark
the presence of a query in server logs. For no explicable reason, the
tests with the marker 'connect2' occur before the tests that use
'connect1' marker.

The attached patch swaps the connection marker strings so that a
reader doesn't have to spend extra deciphering why 'connect2' tests
appear before 'connect1' tests.

Best regards,
Gurjeet https://Gurje.et
Postgres Contributors Team, http://aws.amazon.com

  2. Re: Reorder connection markers in libpq TAP tests

    Gurjeet Singh <gurjeet@singh.im> — 2023-04-21T06:38:25Z

    On Thu, Apr 20, 2023 at 11:00 PM Gurjeet Singh <gurjeet@singh.im> wrote:
>
> Commit 7f5b198 introduced TAP tests that use string literals to mark
> the presence of a query in server logs. For no explicable reason, the
> tests with the marker 'connect2' occur before the tests that use
> 'connect1' marker.
>
> The attached patch swaps the connection marker strings so that a
> reader doesn't have to spend extra deciphering why 'connect2' tests
> appear before 'connect1' tests.

Please see attached v2 of the patch. It now includes same fix in
another TAP tests file.


Best regards,
Gurjeet https://Gurje.et
Postgres Contributors Team, http://aws.amazon.com

  3. Re: Reorder connection markers in libpq TAP tests

    Jelte Fennema-Nio <postgres@jeltef.nl> — 2023-04-21T09:41:50Z

    LGTM I guess this was an unintended leftover from me reordering the tests a
bit in the final stages of getting these patches in.

On Fri, 21 Apr 2023 at 08:38, Gurjeet Singh <gurjeet@singh.im> wrote:

> On Thu, Apr 20, 2023 at 11:00 PM Gurjeet Singh <gurjeet@singh.im> wrote:
> >
> > Commit 7f5b198 introduced TAP tests that use string literals to mark
> > the presence of a query in server logs. For no explicable reason, the
> > tests with the marker 'connect2' occur before the tests that use
> > 'connect1' marker.
> >
> > The attached patch swaps the connection marker strings so that a
> > reader doesn't have to spend extra deciphering why 'connect2' tests
> > appear before 'connect1' tests.
>
> Please see attached v2 of the patch. It now includes same fix in
> another TAP tests file.
>
>
> Best regards,
> Gurjeet https://Gurje.et
> Postgres Contributors Team, http://aws.amazon.com
>

  4. Re: Reorder connection markers in libpq TAP tests

    Daniel Gustafsson <daniel@yesql.se> — 2023-04-21T10:55:44Z

    > On 21 Apr 2023, at 08:38, Gurjeet Singh <gurjeet@singh.im> wrote:
> 
> On Thu, Apr 20, 2023 at 11:00 PM Gurjeet Singh <gurjeet@singh.im> wrote:
>> 
>> Commit 7f5b198 introduced TAP tests that use string literals to mark
>> the presence of a query in server logs. For no explicable reason, the
>> tests with the marker 'connect2' occur before the tests that use
>> 'connect1' marker.
>> 
>> The attached patch swaps the connection marker strings so that a
>> reader doesn't have to spend extra deciphering why 'connect2' tests
>> appear before 'connect1' tests.
> 
> Please see attached v2 of the patch. It now includes same fix in
> another TAP tests file.

-	  'Potentially unsafe test load_balance not enabled in PG_TEST_EXTRA';
+	  'Potentially unsafe test; load_balance not enabled in PG_TEST_EXTRA';

We have this spelling without a ';' in multiple places so I left this alone.

I have applied this version of the patch apart from the above hunk. Thanks!

--
Daniel Gustafsson