Postgres Pro
Downloads
Home   >   mailing lists

Re: [PATCH] Extend ALTER OPERATOR to support adding commutator, negator, hashes, and merges - Mailing list pgsql-hackers

From Christoph Berg
Subject Re: [PATCH] Extend ALTER OPERATOR to support adding commutator, negator, hashes, and merges
Date
Msg-id ZUEoOKHdvzGn5aIc@msg.df7cb.de
Whole thread Raw
In response to Re: [PATCH] Extend ALTER OPERATOR to support adding commutator, negator, hashes, and merges  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: [PATCH] Extend ALTER OPERATOR to support adding commutator, negator, hashes, and merges
List pgsql-hackers
Tree view 
Re: Tom Lane
> Well, the idea was exactly to forbid that sort of setup.

Fwiw, pgsphere has remove the problematic operators now:

https://github.com/postgrespro/pgsphere/commit/e810f5ddd827881b06a92a303c5c9fbf997b892e

> 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?

Hmm, that seems backwards, I'd expect that CREATE might have some
checks that could circumvent using ALTER if I really insisted. If
CREATE can create things that I can't reach by ALTERing existing other
things, that's weird.

Let's keep it like it is now in PG17.

Christoph

pgsql-hackers by date:

Previous
From: Tommy Pavlicek
Date:
Subject: Re: [PATCH] Extend ALTER OPERATOR to support adding commutator, negator, hashes, and merges
Next
From: Nathan Bossart
Date:
Subject: Re: [17] Special search_path names "!pg_temp" and "!pg_catalog"