[talmacschen-arch]

Problem

GetAllViews builds a materialized view's WITH clause by joining pg_class.reloptions verbatim with , , the same way GetTableStorage did for tables (fixed in #127). A reloption value that is not a plain token does not survive the round trip. The prime case is a PAX materialized view, whose minmax_columns/cluster_columns carry a comma-separated column list:

CREATE MATERIALIZED VIEW ... USING pax WITH (minmax_columns=a,b) AS ...
ERROR: unrecognized parameter "b" (SQLSTATE 22023)

The matview is never created and its ALTER ... OWNER then cascades into relation does not exist, so the materialized view is lost on restore. Uppercase or spaced option values break the same way.

This is the materialized-view follow-up noted in #127 — same root cause, different query.

Fix

Render reloptions through a shared helper relOptionsWithClause, used by both the GPDB6 and GPDB7+/Cloudberry matview queries. It single-quotes any value that is not a simple token (not matching ^[a-z0-9_.]+$), covering the comma / uppercase / space cases, and:

• leaves plain-token values (fillfactor=50, autovacuum_enabled=false, ...) byte-for-byte unchanged;
• preserves the original ' WITH (...)' vs '' (no reloptions) behaviour via an explicit NULL check (so a matview without reloptions is not emitted as WITH ());
• preserves option order with WITH ORDINALITY.

Matview data handling is unchanged: matviews are still emitted WITH NO DATA; only the definition is restored (a manual REFRESH MATERIALIZED VIEW repopulates it).

Verification

End to end on a live Apache Cloudberry 2.1.0 (SynxDB4 4.5.4-3) cluster with a rebuilt gpbackup/gprestore:

Object	stock gpbackup 2.2.0	this PR
PAX matview minmax_columns='a,b', cluster_columns='a,b'	❌ WITH (minmax_columns=a,b) → unrecognized parameter "b", matview lost	✅ WITH (minmax_columns='a,b', cluster_columns='a,b', ...), restores cleanly
plain heap matview, no reloptions	✅	✅ still emitted without a WITH clause (no WITH ())

go build ./backup/... and the integration test package compile; added an integration test asserting the quoted WITH clause recreates the matview verbatim (CBDB-gated, with a round-trip CREATE MATERIALIZED VIEW).