On Wed, Oct 22, 2025 at 12:27 PM Chao Li <li.evan.chao@gmail.com> wrote: > > Hi Tim, > > Thanks for working on this. I haven’t finished reviewing the entire patch. But I got a quick question: > > While reviewing a similar patch of pg_get_policy_ddl(), it take the last parameter as a pretty flag. I wonder why pg_get_domain_ddl() doesn’t support an argument for pretty? > >
That's a good point; we'll add pretty printing support for consistency with the other functions. I'll send a new patch in the coming days.
I've already implemented a generic function for pretty-formatted DDL in the ruleutils.c file as part of my pg_get_policy_ddl patch. I suggest reusing it once my patch is accepted and committed by the community.
Best regards, Florin Irion Tim Waizenegger EDB (EnterpriseDB)
