On Mon, Jan 23, 2023 at 3:29 PM Nitin Jadhav <nitinjadhavpostg...@gmail.com> wrote: > > > The v2 patch looks good to me except the comment around > > ConfigOptionIsShowable() which is too verbose. How about just "Return > > whether the GUC variable is visible or not."? > > Sounds good. Updated in the v3 patch attached. > > > I think you can add it to CF, if not done, to not lose track of it. > > Added https://commitfest.postgresql.org/42/4140/
LGTM. I've marked it RfC. -- Bharath Rupireddy PostgreSQL Contributors Team RDS Open Source Databases Amazon Web Services: https://aws.amazon.com
