public inbox for [email protected]  

help / color / mirror / Atom feed

From: Nathan Bossart <[email protected]>
To: Daniel Gustafsson <[email protected]>
Cc: PostgreSQL Hackers <[email protected]>
Subject: Re: Is pg_regress --use-existing used by anyone or is it broken?
Date: Tue, 29 Aug 2023 14:38:27 -0700
Message-ID: <20230829213827.GB2266537@nathanxps13> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>

On Mon, Aug 28, 2023 at 03:11:15PM +0200, Daniel Gustafsson wrote:
> When looking at pg_regress I noticed that the --use-existing support didn't
> seem to work.  ISTM that the removal/creation of test databases and roles
> doesn't run since the conditional is reversed.  There is also no support for
> using a non-default socket directory with PG_REGRESS_SOCK_DIR.  The attached
> hack fixes these and allows the tests to execute for me, but even with that the
> test_setup suite fails due to the tablespace not being dropped and recreated
> like databases and roles.
> 
> Is it me who is too thick to get it working, or is it indeed broken?  If it's
> the latter, it's been like that for a long time which seems to indicate that it
> isn't really used and should probably be removed rather than fixed?
> 
> Does anyone here use it?

I don't think I've ever used it.  AFAICT it was added with hot standby mode
(efc16ea) to support 'make standbycheck', which was removed last year
(4483b2cf).  Unless someone claims to be using it, it's probably fine to
just remove it.

-- 
Nathan Bossart
Amazon Web Services: https://aws.amazon.com

view thread (2+ messages)

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected]
  Subject: Re: Is pg_regress --use-existing used by anyone or is it broken?
  In-Reply-To: <20230829213827.GB2266537@nathanxps13>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox