MIME-Version: 1.0
References: 
 <CAHGQGwGNTaXnBKUV510_P1KwhdbHT+kgZ4zU5njBHy7nCqdhzg@mail.gmail.com>
 <DJCK033ELDM0.24V36DWNB6XH@partin.io>
 <CAHGQGwE95GLD7RZr2g=zTHKJRafKACmKouhb4TTVH155Hy8zKA@mail.gmail.com>
 <DJD69GFRSZ8G.27K1856EEDBF0@partin.io>
 <CAHGQGwFLXCAxKEyvvaTLadHZ8k4DFmjJopnN+Vy-NjtfKoM7dw@mail.gmail.com>
 <CAA4eK1JOo0aJRhFHNWpj3hMwaTtNOopY34f1Lh_QD=z=+DrzWQ@mail.gmail.com>
 <CAHGQGwEA6PMAE0aMx4MFqj9d+q2Q835gfUyTSJjAkoMwtWB77A@mail.gmail.com>
 <CAA4eK1+C+mG6ihu_Cs3KjMw9DJgUneSXy5k0HsXyoE1EeWQ1gQ@mail.gmail.com>
 <CAHGQGwHuuffj_js4n2g2M07vpkbun8Aj_4+o4ALK=rQpBfwk7w@mail.gmail.com>
 <CAA4eK1+UU-MLCWiO0Yf8WPakt6tSK2YsFDCQnzt6616+jy2CRw@mail.gmail.com>
 <CAHGQGwH1Ssze3XM6wjoTjSLVOR041c6xP+vsdLP951=w8oG8bA@mail.gmail.com>
 <CAHGQGwEcGpnHWw1uwsWkgVwKv5cwysKfir2yXDQnN1aVOAhCdQ@mail.gmail.com>
 <CALDaNm2o9o-QOArgO3Dmz-Kb_goRzNse41G4CQh8SgJWrU2WLQ@mail.gmail.com>
 <CAHGQGwE9AOvhqPOx7J=zPug5s90FhNVPH8m0m3AVJVzjG8npBQ@mail.gmail.com>
In-Reply-To: 
 <CAHGQGwE9AOvhqPOx7J=zPug5s90FhNVPH8m0m3AVJVzjG8npBQ@mail.gmail.com>
From: vignesh C <vignesh21@gmail.com>
Date: Mon, 29 Jun 2026 21:31:14 +0530
Message-ID: 
 <CALDaNm1DWVcFA10ULBaBVJg7sAGnDHOzDbAmFGTWj5mEQ3bPUw@mail.gmail.com>
Subject: Re: Fix publisher-side sequence permission reporting
To: Fujii Masao <masao.fujii@gmail.com>
Cc: Amit Kapila <amit.kapila16@gmail.com>, Tristan Partin <tristan@partin.io>,
	PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Archived-At: 
 <https://www.postgresql.org/message-id/CALDaNm1DWVcFA10ULBaBVJg7sAGnDHOzDbAmFGTWj5mEQ3bPUw%40mail.gmail.com>
Precedence: bulk

On Fri, 26 Jun 2026 at 12:34, Fujii Masao <masao.fujii@gmail.com> wrote:
>
> On Wed, Jun 24, 2026 at 7:49=E2=80=AFPM vignesh C <vignesh21@gmail.com> w=
rote:
> > Thanks for the patch. I verified that it fixes the issue with reading
> > unlogged sequences on a promoted standby.
>
> Thanks for testing!
>
>
> > Do you think it would be worthwhile to add a test for this scenario,
> > or do you feel the additional test is not necessary in this case?
>
> I think it's worth adding a test for this scenario, so I've added one to
> the patch. The test uses nextval() to read the unlogged sequence
> instead of pg_get_sequence_data(), since this patch needs to be
> backpatched to v15, while pg_get_sequence_data() was introduced in
> v19.

Thanks for the updated patches.  I verified that the issue occurs with
nextval() on all supported branches up to v15, where unlogged
sequences are available. The corresponding back patches apply cleanly
to their respective branches and fix the issue.  I also checked other
unlogged objects but couldn't reproduce a similar issue with them.

Overall, the patches look good to me.

Regards,
Vignesh