Re: Why do pg_upgrade's test use the serial schedule?

From: Andres Freund <andres(at)anarazel(dot)de>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-hackers(at)postgresql(dot)org, Bruce Momjian <bruce(at)momjian(dot)us>, Peter Eisentraut <peter_e(at)gmx(dot)net>
Subject: Re: Why do pg_upgrade's test use the serial schedule?
Date: 2018-11-05 22:00:30
Message-ID: 20181105220030.qmzdthtkabuomiy4@alap3.anarazel.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 2018-11-05 16:32:20 -0500, Tom Lane wrote:
> Andres Freund <andres(at)anarazel(dot)de> writes:
> > On 2018-11-05 16:10:28 -0500, Tom Lane wrote:
> >> Andres Freund <andres(at)anarazel(dot)de> writes:
> >>> ISTM we also should disable fsyncs for the postmaster instances. Wins
> >>> another ~5s on my laptop, and I suspect it makes a larger difference on
> >>> some of the buildfarm animals.
>
> >> Buildfarm did that long ago.
>
> > I don't think it did for pg_upgrade's test.sh?
>
> It's injected via the TEMP_CONFIG env variable.

Ah, thanks. Is there any good reason to not instead have it in the
script? Doesn't strike me as great that it slows down normal regression
runs during development? Injecting TEMP_CONFIG w/ fsync=off globally
for those (rather than in a single test wrapper), could counteract
explicit tests where fsync is wanted - beside the fact that one has to
do so manually. Normal regression tests have long done so.

Greetings,

Andres Freund

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2018-11-05 22:08:12 Re: Why do pg_upgrade's test use the serial schedule?
Previous Message Corey Huinker 2018-11-05 21:53:19 Re: partitioned tables referenced by FKs