Re: Int_aggregate/pgstattuple minor fix - Mailing list pgsql-patches

From Tom Lane
Subject Re: Int_aggregate/pgstattuple minor fix
Date
Msg-id 10015.1096645197@sss.pgh.pa.us
Whole thread Raw
In response to Int_aggregate/pgstattuple minor fix  ("Dave Page" <dpage@vale-housing.co.uk>)
List pgsql-patches
"Dave Page" <dpage@vale-housing.co.uk> writes:
> The patch below removes attempts to drop objects before creating them
> from the int_aggregate and pgstattuple SQL scripts. This causes the
> script to fail on databases where the affected objects don't already
> exist when the script is executed by the Win32 installer, pgAdmin or any
> other interface that executes scripts 'as-is'. None of the other contrib
> scripts appear do this.

Seems reasonable.  You could also make the argument that the DROPs
actually risk dropping the wrong object, since they could find a
matching name in the schema search path that is not in the current
schema (and thus not really a conflict).

            regards, tom lane

pgsql-patches by date:

Previous
From: Tom Lane
Date:
Subject: Re: Preliminary patch for FRONTEND
Next
From: Peter Eisentraut
Date:
Subject: Re: LDFLAGS overriding