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
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 |