pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Fujii Masao <fujii@postgresql.org>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Add a new helper function wait_for_file() to Utils.pm.
Date: Fri, 16 Jan 2026 03:39:58 +0000
Message-ID: <E1vgagn-000jfq-2u@gemulon.postgresql.org> (raw)

Add a new helper function wait_for_file() to Utils.pm.

wait_for_file() waits for the contents of a specified file, starting at an
optional offset, to match a given regular expression. If no offset is
provided, the entire file is checked. The function times out after
$PostgreSQL::Test::Utils::timeout_default seconds. It returns the total
file length on success.

The existing wait_for_log() function contains almost identical logic, but
is limited to reading the cluster's log file. This commit also refactors
wait_for_log() to call wait_for_file() instead, avoiding code duplication.

This helper will be used by upcoming changes.

Suggested-by: Mircea Cadariu <cadariu.mircea@gmail.com>
Author: Fujii Masao <masao.fujii@gmail.com>
Reviewed-by: Mircea Cadariu <cadariu.mircea@gmail.com>
Reviewed-by: Chao Li <li.evan.chao@gmail.com>
Discussion: https://postgr.es/m/CAHGQGwFeTymZQ7RLvMU6WuDGar8bUQCazg=VOfA-9GeBkg-FzA@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/0b10969db610c607227b23710d556b595d7eb34a

Modified Files
--------------
src/test/perl/PostgreSQL/Test/Cluster.pm | 20 ++--------------
src/test/perl/PostgreSQL/Test/Utils.pm   | 39 ++++++++++++++++++++++++++++++++
2 files changed, 41 insertions(+), 18 deletions(-)



Message-ID: <E1vgagn-000jfq-2u@gemulon.postgresql.org>
Permalink:  ../E1vgagn-000jfq-2u@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1vgagn-000jfq-2u@gemulon.postgresql.org

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: fujii@postgresql.org, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Add a new helper function wait_for_file() to Utils.pm.
  In-Reply-To: <E1vgagn-000jfq-2u@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox