Re: Improve error reporting in 027_stream_regress test

From: Andres Freund <andres(at)anarazel(dot)de>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Improve error reporting in 027_stream_regress test
Date: 2025-07-29 01:56:28
Message-ID: pilcuplleg3o5l4zzjerpz4xthqasmu7b3tenp6yuj2l7sfi2e@w7za3ejtcz6q
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
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

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2025-07-29 02:21:41 Broken ./configure checks for __cpuid() and __cpuidex()
Previous Message Hayato Kuroda (Fujitsu) 2025-07-29 01:53:00 RE: 024_add_drop_pub.pl might fail due to deadlock