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

PG17 compatibility: add helper function for EXPLAIN diffs in scalar subquery output #7757

Merged
merged 1 commit into from
Nov 21, 2024

Conversation

colm-mchugh
Copy link
Contributor

PG17 changed how scalar subquery outputs appear in EXPLAIN output (*). This commit changes impacted regress goldfiles to the PG17 format, and adds a helper function to covert pre-PG17 plans to the PG17 format. The conversion is required when testing Citus on pgversions prior to 17. The helper function can and should be removed when 17 becomes the minimum supported version.

(*) https://git.postgresql.org/gitweb/?p=postgresql.git;a=commitdiff;h=fd0398fcb

Copy link

codecov bot commented Nov 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (release-13.0@680c23f). Learn more about missing BASE report.

Additional details and impacted files
@@               Coverage Diff               @@
##             release-13.0    #7757   +/-   ##
===============================================
  Coverage                ?   89.64%           
===============================================
  Files                   ?      274           
  Lines                   ?    59583           
  Branches                ?     7436           
===============================================
  Hits                    ?    53413           
  Misses                  ?     4038           
  Partials                ?     2132           
---- 🚨 Try these New Features:

Copy link
Member

@naisila naisila left a comment

Choose a reason for hiding this comment

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

Great fix, thanks Colm.
Let's rebase to release-13.0 before merging.

@naisila naisila changed the base branch from naisila/pg17_support to release-13.0 November 21, 2024 13:43
@naisila naisila changed the title PG17 support: change regress goldfiles for how PG17 EXPLAINs scalar subquery ouput PG17 compatibility: change regress goldfiles for how PG17 EXPLAINs scalar subquery ouput Nov 21, 2024
@naisila naisila changed the title PG17 compatibility: change regress goldfiles for how PG17 EXPLAINs scalar subquery ouput PG17 compatibility: add helper function for PG17 EXPLAIN diffs in scalar subquery output Nov 21, 2024
The relevant PG commit is:
https://git.postgresql.org/gitweb/?p=postgresql.git;a=commitdiff;h=fd0398fcb

This commit changes impacted regress goldfiles to the PG17 format, and
adds a helper function to covert pre-PG17 plans to the PG17 format.
The conversion is required when pgversoin < 17. It can and should be
removed when 17 becomes the minimum supported version.
@naisila naisila changed the title PG17 compatibility: add helper function for PG17 EXPLAIN diffs in scalar subquery output PG17 compatibility: add helper function for EXPLAIN diffs in scalar subquery output Nov 21, 2024
@naisila naisila merged commit 7e701be into release-13.0 Nov 21, 2024
121 checks passed
@naisila naisila deleted the cmchugh/pg17-Explain-InitPlans branch November 21, 2024 19:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants