Skip to content

Drop obsolete compatibility @({bigint,spoint}, smoc) operators #92

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 27, 2023

Conversation

df7cb
Copy link
Contributor

@df7cb df7cb commented Oct 26, 2023

These were required by GAVO DaCHS, but are not used anymore. Furthermore, PG17 will forbid declaring commutators and negators relations with operators that themselves have relations to different operators.

We delete pgs_moc_compat.sql.in and remove it from the old pg_sphere--1.1.5beta2gavo--1.1.5beta4gavo.sql upgrade script. This is ok to do since dropping the operators uses "if exists".

Discussion: https://www.postgresql.org/message-id/flat/ZTfL1G1fBPQHVPWV%40msg.df7cb.de#ce3801c084bcf05b6b976cb2d0e25b2d

Close #85.

@df7cb
Copy link
Contributor Author

df7cb commented Oct 26, 2023

Sorry, I had typoed the "git push --force" command and mistakenly deleted the branch which auto-closed #91. This is the same patch rebased to current master.

Copy link
Contributor

@esabol esabol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

These were required by GAVO DaCHS, but are not used anymore.
Furthermore, PG17 will forbid declaring commutators and negators
relations with operators that themselves have relations to different
operators.

We delete pgs_moc_compat.sql.in and remove it from the old
pg_sphere--1.1.5beta2gavo--1.1.5beta4gavo.sql upgrade script. This is ok
to do since dropping the operators uses "if exists".

Discussion: https://www.postgresql.org/message-id/flat/ZTfL1G1fBPQHVPWV%40msg.df7cb.de#ce3801c084bcf05b6b976cb2d0e25b2d

Close postgrespro#85.
@vitcpp vitcpp merged commit 6f19a10 into postgrespro:master Oct 27, 2023
@vitcpp
Copy link
Contributor

vitcpp commented Oct 27, 2023

@df7cb Thank you for the PRs! I propose to make the new release at the beginning of the next week if no new PRs will appear.

@df7cb df7cb deleted the drop-compat-ops branch October 30, 2023 09:51
@df7cb
Copy link
Contributor Author

df7cb commented Oct 30, 2023

I have another indexing idea in the pipeline that would make sense including with 1.4, so please wait a bit. I should be able to submit it this week.

@df7cb
Copy link
Contributor Author

df7cb commented Oct 31, 2023

@vitcpp I need more time, so please don't wait with 1.4 and go on with a release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Operator @ is incompatible with PG17
3 participants