On Wed, 22 Oct, 2025, 17:30 Tim Waizenegger, < [email protected]> wrote:
> On Wed, Oct 22, 2025 at 12:27 PM Chao Li <[email protected]> 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) > > >
