Re: Implement generalized sub routine find_in_log for tap test - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: Implement generalized sub routine find_in_log for tap test
Date
Msg-id ZHu852YlGAk7AVk0@paquier.xyz
Whole thread Raw
In response to Re: Implement generalized sub routine find_in_log for tap test  (vignesh C <vignesh21@gmail.com>)
Responses Re: Implement generalized sub routine find_in_log for tap test
Re: Implement generalized sub routine find_in_log for tap test
List pgsql-hackers
On Mon, May 29, 2023 at 07:49:52AM +0530, vignesh C wrote:
> Thanks for the comment, the attached v3 version patch has the changes
> for the same.

-if (find_in_log(
-        $node, $log_offset,
-        qr/peer authentication is not supported on this platform/))
+if ($node->log_contains(
+        qr/peer authentication is not supported on this platform/),
+        $log_offset,)

This looks like a typo to me, the log offset is eaten.

Except of that, I am on board with log_contains().

There are two things that bugged me with the refactoring related to
connect_ok and connect_fails:
- check_connect_log_contents() is a name too complicated.  While
looking at that I have settled to a simpler log_check(), as we could
perfectly use that for something else than connections as long as we
want to check multiple patterns at once.
- The refactoring of the documentation for the routines of Cluster.pm
became incorrect.  For example, the patch does not list anymore
log_like and log_unlike for connect_ok.

With all that in mind I have hacked a few adjustments in a 0003,
though I agree with the separation between 0001 and 0002.

033_replay_tsp_drops and 019_replslot_limit are not new to v16, but
003_peer.pl and 035_standby_logical_decoding.pl, making the number of
places where find_in_log() exists twice as much.  So I would be
tempted to refactor these tests in v16.  Perhaps anybody from the RMT
could comment?  We've usually been quite flexible with the tests even
in beta.

Thoughts?
--
Michael

Attachment

pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: Prevent psql \watch from running queries that return no rows
Next
From: Michael Paquier
Date:
Subject: Re: [PATCH] Slight improvement of worker_spi.c example