Re: pgsql: Adjust interaction of CREATEROLE with role properties. - Mailing list pgsql-committers

From Tom Lane
Subject Re: pgsql: Adjust interaction of CREATEROLE with role properties.
Date
Msg-id 3255990.1674789499@sss.pgh.pa.us
Whole thread Raw
In response to Re: pgsql: Adjust interaction of CREATEROLE with role properties.  (Michael Paquier <michael@paquier.xyz>)
Responses Re: pgsql: Adjust interaction of CREATEROLE with role properties.  (Andrew Dunstan <andrew@dunslane.net>)
List pgsql-committers
Michael Paquier <michael@paquier.xyz> writes:
> On Thu, Jan 26, 2023 at 09:08:23PM -0500, Tom Lane wrote:
>> Is it worth checking for leftover regress_xxx tablespaces as well as
>> roles?

> Guess so.  If this is to be applied to everything that fails under the
> naming restrictions, there could be a point in doing the same for
> databases, subscriptions and replication origins.  Now the argument
> has less weight for these object types, surely, compared to roles and
> tbspaces.

By the time we've spun up a new backend, we might as well do more
than one query, so it seems like we might as well check everything
that's globally visible.

However ... this'd only cover mistakes of this kind in the core
regression tests.  Is there a way to cover contrib's installcheck
(without adding an additional psql run to each module's tests)?

Maybe we need to enforce this at some other level than the tests
themselves, perhaps in pg_regress?

            regards, tom lane



pgsql-committers by date:

Previous
From: David Rowley
Date:
Subject: pgsql: Teach planner about more monotonic window functions
Next
From: Peter Eisentraut
Date:
Subject: pgsql: doc: Adjust a few more references to "postmaster"