Re: Add find_in_log() and advance_wal() perl functions to core test framework (?)

From: Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>
To: alvherre(at)alvh(dot)no-ip(dot)org
Cc: andrew(at)dunslane(dot)net, bharath(dot)rupireddyforpostgres(at)gmail(dot)com, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: Add find_in_log() and advance_wal() perl functions to core test framework (?)
Date: 2022-08-24 01:12:23
Message-ID: 20220824.101223.1719168738216544953.horikyota.ntt@gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

At Tue, 16 Aug 2022 18:40:49 +0200, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> wrote in
> On 2022-Aug-16, Andrew Dunstan wrote:
>
> > I don't think there's a hard and fast rule about it. Certainly the case
> > would be more compelling if the functions were used across different TAP
> > suites. The SSL suite has suite-specific modules. That's a pattern also
> > worth considering. e.g something like.
> >
> >     use FindBin qw($Bin);
> >     use lib $Bin;
> >     use MySuite;
> >
> > and then you put your common routines in MySuite.pm in the same
> > directory as the TAP test files.
>
> Yeah, I agree with that for advance_wal. Regarding find_in_log, that
> one seems general enough to warrant being in Cluster.pm -- consider
> issues_sql_like, which also slurps_file($log). That could be unified a
> little bit, I think.

+1

regards.

--
Kyotaro Horiguchi
NTT Open Source Software Center

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Kyotaro Horiguchi 2022-08-24 01:48:01 Re: add checkpoint stats of snapshot and mapping files of pg_logical dir
Previous Message Thomas Munro 2022-08-24 00:51:02 Re: logical decoding and replication of sequences