citusdata / citus

Distributed PostgreSQL as an extension
https://www.citusdata.com
GNU Affero General Public License v3.0
10.63k stars 670 forks source link

PG17 compatibility: ruleutils #7725

Closed naisila closed 1 week ago

naisila commented 1 week ago

PG17 compatibility - Part 2

https://github.com/citusdata/citus/pull/7699 was the first PG17 compatibility PR merged to main branch, which provided ONLY successful Citus compilation with PG17.0.

This PR, consider it as Part 2, provides ruleutils changes for PG17. Ruleutils changes is the first thing we should merge, after successful build. It's the core for deparsing logic in Citus.

Question: How do we add ruleutils changes?

Answer:

Notes for reviewer

codecov[bot] commented 1 week ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

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

Additional details and impacted files ```diff @@ Coverage Diff @@ ## release-13.0 #7725 +/- ## =============================================== Coverage ? 89.64% =============================================== Files ? 274 Lines ? 59583 Branches ? 7436 =============================================== Hits ? 53414 Misses ? 4038 Partials ? 2131 ```