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

Fix Isolation Test Failure in isolation_database_cmd_from_any_node in PG17 #7713

Open m3hm3t opened 1 month ago

m3hm3t commented 1 month ago

This PR resolves the failure in the isolation_database_cmd_from_any_node test by updating the object class (oclass_database) value based on the PostgreSQL version. The oclass_database value has changed across versions:

Changes include:

This ensures compatibility with PostgreSQL versions 17, preventing test failures related to incorrect object class identification.

codecov[bot] commented 1 month ago

Codecov Report

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

Project coverage is 89.67%. Comparing base (6c12b10) to head (761d481).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## naisila/pg17_support #7713 +/- ## ===================================================== Coverage 89.67% 89.67% ===================================================== Files 283 283 Lines 60621 60622 +1 Branches 7551 7551 ===================================================== + Hits 54362 54363 +1 Misses 4106 4106 Partials 2153 2153 ```
naisila commented 1 week ago

Let me also finish my review in this PR before merging it. I will post my review soon.