Skip to content
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

Fix ABI issue when upgrading PG from 15.3 to 15.4+ #7589

Merged
merged 1 commit into from
Jan 14, 2025

Conversation

erimatnor
Copy link
Contributor

@erimatnor erimatnor commented Jan 13, 2025

A previous change (commit 8d29760) added a workaround for a PostgreSQL bug related to table access methods that was fixed in PG 15.4. The workaround does a build-time check for the PG version to apply the fix, but this won't work across PG server upgrades.

To avoid raising an error when running an extension compiled on PG < 15.4 on PG 15.4+, add also a runtime check.

Disable-check: force-changelog-file

Copy link

codecov bot commented Jan 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.30%. Comparing base (59f50f2) to head (0e2c16c).
Report is 692 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7589      +/-   ##
==========================================
+ Coverage   80.06%   82.30%   +2.24%     
==========================================
  Files         190      238      +48     
  Lines       37181    43790    +6609     
  Branches     9450    10995    +1545     
==========================================
+ Hits        29770    36043    +6273     
- Misses       2997     3407     +410     
+ Partials     4414     4340      -74     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

A previous change (commit 8d29760) added a workaround for a PostgreSQL
bug related to table access methods that was fixed in PG 15.4. The
workaround does a build-time check for the PG version to apply the
fix, but this won't work across PG server upgrades.

To avoid raising an error when running an extension compiled on PG <
15.4 on PG 15.4+, add also a runtime check.
@erimatnor erimatnor force-pushed the fix-abi-issue-for-pg15 branch from 1c8797a to 0e2c16c Compare January 13, 2025 15:46
@erimatnor erimatnor requested a review from mkindahl January 13, 2025 15:46
@erimatnor
Copy link
Contributor Author

I triggered an ABI run for this PR here: https://github.com/timescale/timescaledb/actions/runs/12750725375

@erimatnor erimatnor added the ci label Jan 13, 2025
@erimatnor erimatnor marked this pull request as ready for review January 13, 2025 15:48
Copy link

@gayyappan, @svenklemm: please review this pull request.

Powered by pull-review

@erimatnor erimatnor merged commit 6623eb6 into timescale:main Jan 14, 2025
51 of 53 checks passed
@erimatnor erimatnor deleted the fix-abi-issue-for-pg15 branch January 14, 2025 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants