Re: [PATCH] Add pg_get_policy_ddl() function to reconstruct CREATE POLICY statement

Thanks, Mark, for your review comments and the updated patch.

I’ve incorporated your changes and prepared a combined v5 patch. The v5
patch is attached for further review.

On Mon, Oct 27, 2025 at 10:15 PM Mark Wong <markwkm(at)gmail(dot)com> wrote:

> Hi everyone,
>
> On Thu, Oct 16, 2025 at 01:47:53PM +0530, Akshay Joshi wrote:
> >
> >
> > On Wed, Oct 15, 2025 at 10:55 PM Álvaro Herrera <alvherre(at)kurilemu(dot)de>
> wrote:
> >
> > Hello,
> >
> > I have reviewed this patch before and provided a number of comments
> that
> > have been addressed by Akshay (so I encourage you to list my name and
> > this address in a Reviewed-by trailer line in the commit message).
> One
> > thing I had not noticed is that while this function has a "pretty"
> flag,
> > it doesn't use it to pass anything to pg_get_expr_worker()'s
> prettyFlags
> > argument, and I think it should -- probably just
> >
> > prettyFlags = GET_PRETTY_FLAGS(pretty);
> >
> > same as pg_get_querydef() does.
>
> Kinda sorta similar thought, I've noticed some existing functions like
> pg_get_constraintdef make the "pretty" flag optional, so I'm wondering
> if that scheme is also preferred here.
>
> I've attached a small diff to the original
> 0001-Add-pg_get_policy_ddl-function-to-reconstruct-CREATE.patch to
> illustrate the additional work to follow suit, if so desired.
>
> Regards,
> Mark
> --
> Mark Wong <markwkm(at)gmail(dot)com>
> EDB https://enterprisedb.com
>