Re: pgsql: Move tablespace path re-creation from the makefiles to pg_regres

From: Christoph Berg <myon(at)debian(dot)org>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Move tablespace path re-creation from the makefiles to pg_regres
Date: 2021-03-24 09:56:29
Message-ID: YFsMzeLE9iiuKJ2a@msg.df7cb.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

Re: Michael Paquier
> So you basically mimicked the makefile rule that this commit removed
> into your own test suite. Reverting the change does not really help,
> because we'd be back to square one where there would be problems in
> parallel runs for developers. Saying that, I would not mind adding an
> option to pg_regress to control if this cleanup code is triggered or
> not, say something like --no-tablespace-cleanup? Then, you could just
> pass down the option by yourself before creating your tablespace path
> as you wish.

I don't think adding more snowflake code just for this use case makes
sense, so I can stick to my workaround.

I just wanted to point out that the only thing preventing the core
testsuite from being run as a true client app is this tablespace
thing, which might be a worthwhile fix on its own.

Maybe creating the tablespace directory from within the testsuite
would suffice?

CREATE TABLE foo (t text);
COPY foo FROM PROGRAM 'mkdir @testtablespace@';
CREATE TABLESPACE regress_tblspace LOCATION '@testtablespace@';

Christoph

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Robert Haas 2021-03-24 12:05:34 Re: pgsql: Add a new GUC and a reloption to enable inserts in parallel-mode
Previous Message Peter Eisentraut 2021-03-24 07:02:48 pgsql: Improve an error message

Browse pgsql-hackers by date

  From Date Subject
Next Message Amit Kapila 2021-03-24 10:06:25 Re: Replication slot stats misgivings
Previous Message Fujii Masao 2021-03-24 09:55:05 Re: Nicer error when connecting to standby with hot_standby=off