From: | Justin Pryzby <pryzby(at)telsasoft(dot)com> |
---|---|
To: | David Christensen <david(dot)christensen(at)crunchydata(dot)com> |
Cc: | Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com>, Ian Lawrence Barwick <barwick(at)gmail(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>, Matthias van de Meent <boekewurm+postgres(at)gmail(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: [PATCH] Teach pg_waldump to extract FPIs from the WAL |
Date: | 2022-11-11 14:15:04 |
Message-ID: | 20221111141504.GF16809@telsasoft.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Nov 09, 2022 at 02:37:29PM -0600, David Christensen wrote:
> On Wed, Nov 9, 2022 at 2:08 PM David Christensen <david(dot)christensen(at)crunchydata(dot)com> wrote:
> > Justin sez:
> > > I was wondering if there's any reason to do "CREATE DATABASE". The vast
> > > majority of TAP tests don't.
> > >
> > > $ git grep -ho 'safe_psql[^ ]*' '*pl' |sort |uniq -c |sort -nr |head
> > > 1435 safe_psql('postgres',
> > > 335 safe_psql(
> > > 23 safe_psql($connect_db,
> >
> > If there was a reason, I don't recall offhand; I will test removing it
> > and if things still work will consider it good enough.
>
> Things blew up when I did that; rather than hunt it down, I just left it in. :-)
> +$primary->safe_psql('db1', <<EOF);
It worked for me when I removed the 3 references to db1.
That's good for efficiency of the test.
> +my $blocksize = 8192;
I think this should be just "my $blocksize;" rather than setting a value
which is later overwriten.
--
Justin
From | Date | Subject | |
---|---|---|---|
Next Message | David Christensen | 2022-11-11 14:27:51 | Re: [PATCH] Teach pg_waldump to extract FPIs from the WAL |
Previous Message | Peter Eisentraut | 2022-11-11 14:08:16 | Re: Privileges on PUBLICATION |