Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

df7cb
Copy link
Contributor

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
ContributorAuthor

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

@esabolesabol 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 negatorsrelations with operators that themselves have relations to differentoperators.We delete pgs_moc_compat.sql.in and remove it from the oldpg_sphere--1.1.5beta2gavo--1.1.5beta4gavo.sql upgrade script. This is okto do since dropping the operators uses "if exists".Discussion:https://www.postgresql.org/message-id/flat/ZTfL1G1fBPQHVPWV%40msg.df7cb.de#ce3801c084bcf05b6b976cb2d0e25b2dClosepostgrespro#85.
@vitcppvitcpp merged commit6f19a10 intopostgrespro:masterOct 27, 2023
@vitcpp
Copy link
Contributor

@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.

esabol reacted with thumbs up emoji

@df7cbdf7cb deleted the drop-compat-ops branchOctober 30, 2023 09:51
@df7cb
Copy link
ContributorAuthor

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.

vitcpp reacted with thumbs up emoji

@df7cb
Copy link
ContributorAuthor

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

vitcpp reacted with thumbs up emoji

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@vitcppvitcppvitcpp approved these changes

@esabolesabolesabol approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Operator@ is incompatible with PG17
3 participants
@df7cb@vitcpp@esabol

[8]ページ先頭

©2009-2025 Movatter.jp