PG17 Add Regression Test for ALTER TABLE ... SET ACCESS METHOD DEFAULT

[m3hm3t]

This PR introduces and enforces an error check preventing ALTER TABLE ... SET ACCESS METHOD DEFAULT on both Citus local tables (added via citus_add_local_table_to_metadata) and distributed/partitioned distributed tables. The regression tests now demonstrate that each table type raises an error advising users to explicitly specify an access method, rather than relying on DEFAULT. This ensures consistent behavior across local and distributed environments in Citus.

postgres/postgres@d61a6cad6

[codecov]

Codecov Report

Attention: Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.

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

Additional details and impacted files

@@               Coverage Diff               @@
##             release-13.0    #7803   +/-   ##
===============================================
  Coverage                ?   89.64%           
===============================================
  Files                   ?      274           
  Lines                   ?    59726           
  Branches                ?     7452           
===============================================
  Hits                    ?    53540           
  Misses                  ?     4056           
  Partials                ?     2130           

[naisila]

We have a problem here because default access method may be different in the cluster nodes. Steps to reproduce below:

CREATE ACCESS METHOD heap2 TYPE TABLE HANDLER heap_tableam_handler;

CREATE TABLE dist_table (id SERIAL PRIMARY KEY, data TEXT) USING heap;
SELECT create_distributed_table('dist_table', 'id');

SET default_table_access_method = 'heap2';

-- default in coordinator is heap2, default in workers is heap
ALTER TABLE dist_table SET ACCESS METHOD DEFAULT;

SELECT c.relname, am.amname FROM pg_class c, pg_am am
WHERE c.relam = am.oid and c.relname IN ('dist_table', 'ref_table');
  relname   | amname
------------+--------
 dist_table | heap2
(1 row)

\c - - - :worker_1_port
SELECT c.relname, am.amname FROM pg_class c, pg_am am
WHERE c.relam = am.oid and c.relname IN ('dist_table', 'ref_table');
  relname   | amname
------------+--------
 dist_table | heap
(1 row)

The problem is that the alter table command is distributed as it is. One way to fix this is to propagate the alter table command by replacing "default" with the actual access method name. This requires code changes

[naisila]

LGTM 🚀

Rebase to release-13.0 and merge after all tests have passed (except packaging images tests)