From: Tom Lane <tgl@sss.pgh.pa.us>
To: Christoph Berg <myon@debian.org>
cc: Tommy Pavlicek <tommypav122@gmail.com>,
        Dagfinn Ilmari =?iso-8859-1?Q?Manns=E5ker?= <ilmari@ilmari.org>,
        pgsql-hackers@lists.postgresql.org,
        Tomas Vondra <tomas.vondra@enterprisedb.com>,
        jian.universality@gmail.com
Subject: Re: [PATCH] Extend ALTER OPERATOR to support adding commutator,
 negator, hashes, and merges
In-reply-to: <ZTfjXzOKFI9AawJF@msg.df7cb.de>
References: <4082750.1687455374@sss.pgh.pa.us>
 <CAEhP-W_xGQVmiEJE_x0x5bbDSwsmaN1TQEawNadiMvQj2XK2Sg@mail.gmail.com>
 <15382.1687519306@sss.pgh.pa.us>
 <CAEhP-W-bY5CTXgrsH_J=NyKJpqMOgUPqMQ9-oTtx2N+zND1ZuA@mail.gmail.com>
 <1092363.1695932314@sss.pgh.pa.us>
 <CAEhP-W8MjoA0yDCipp=FgZvoQA-OVjQws3-XO6bpCWXunO8t8A@mail.gmail.com>
 <447166.1696969927@sss.pgh.pa.us>
 <CAEhP-W_1bMTLan2LtzS+xr=b-Wep56gw+OoirABQ8b2aOw+5yg@mail.gmail.com>
 <ZTfL1G1fBPQHVPWV@msg.df7cb.de> <560086.1698160614@sss.pgh.pa.us>
 <ZTfjXzOKFI9AawJF@msg.df7cb.de>
Comments: In-reply-to Christoph Berg <myon@debian.org>
	message dated "Tue, 24 Oct 2023 17:31:43 +0200"
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-ID: <570832.1698162135.1@sss.pgh.pa.us>
Date: Tue, 24 Oct 2023 11:42:15 -0400
Message-ID: <570833.1698162135@sss.pgh.pa.us>
Archived-At: 
 <https://www.postgresql.org/message-id/570833.1698162135%40sss.pgh.pa.us>
Precedence: bulk

Christoph Berg <myon@debian.org> writes:
> Anyway, if this doesn't raise any "oh we didn't think of this"
> concerns, we'll just remove the old operators in pgsphere.

Well, the idea was exactly to forbid that sort of setup.
However, if we get sufficient pushback maybe we should
reconsider --- for example, maybe it'd be sane to enforce
the restriction in ALTER but not CREATE?

I'm inclined to wait and see if there are more complaints.

			regards, tom lane