Re: Bogus documentation for bogus geometric operators
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Pavel Borisov <pashkin.elfe@gmail.com>
Cc: Emre Hasegeli <emre@hasegeli.com>, Michael Paquier <michael@paquier.xyz>,
PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2020-11-22T23:52:26Z
Lists: pgsql-hackers
Attachments
- v6-0001-Deprecate-and-replace-operators-for-points.patch (text/x-diff) patch v6-0001
Pavel Borisov <pashkin.elfe@gmail.com> writes: >> undocumented. Maybe instead of removing, change the text to be >> "Deprecated, use the equivalent XXX operator instead." Or we could >> add a footnote similar to what was there for a previous renaming: > The problem that this new <<| is equivalent to <^ only for points (To > recap: the source of a problem is the same name of <^ operator for points > and boxes with different meaning for these types). I don't think it's that hard to be clear; see proposed wording below. The other loose end is that I don't think we can take away the opclass entries for the old spellings, unless we're willing to visibly break people's queries by removing those operator names altogether. That doesn't seem like it'll fly when we haven't even deprecated the old names yet. So for now, we have to support both names in the opclasses. I extended the patch to do that. This version seems committable to me --- any thoughts? regards, tom lane
Commits
-
Rename the "point is strictly above/below point" comparison operators.
- 0cc993278888 14.0 landed
-
Remove deprecated containment operators for built-in types
- 2f70fdb0644c 14.0 cited