Re: Improve error reporting in 027_stream_regress test

Andres Freund <andres@anarazel.de>

From: Andres Freund <andres@anarazel.de>
To: Michael Paquier <michael@paquier.xyz>
Cc: Nazir Bilal Yavuz <byavuz81@gmail.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2025-07-29T01:56:28Z
Lists: pgsql-hackers
Hi,

On 2025-07-29 10:46:16 +0900, Michael Paquier wrote:
> On Mon, Jul 28, 2025 at 03:34:06PM +0300, Nazir Bilal Yavuz wrote:
> > I wanted to show what is in my mind, v4 is attached. Summary is:
> > 
> > - 0001 introduces the read_file_ends() function, which reads lines
> > from either the beginning or end of a given file. It includes a
> > force_line_count argument that, when set to true, ensures that the
> > specified number of lines is read from the file regardless of whether
> > the PG_TEST_FILE_READ_LINES environment variable is set.
> 
> Hmm.  I am really wondering if 'head' and 'tail' should just be
> shortcuts of some regexp patterns based on ^ and $, and that we could
> allow the subroutine to use a regexp for the pattern matching.  I'll
> look at that in details, but it may take me a couple of days before
> getting down to it.

I don't really get what the point of designing that mechanism is before we
have a usecase. If we need it, we can expand it at that time.

Greetings,

Andres Freund



Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →
  1. Improve error reporting of recovery test 027_stream_regress

  2. Add PostgreSQL::Test::Cluster::read_head_tail() helper to PostgreSQL/Utils.pm

  3. Handle timeout in PostgreSQL::Test::Cluster::is_alive()

  4. Check status of nodes after regression test run in 027_stream_regress

  5. Add PostgreSQL::Test::Cluster::is_alive()