On Sat, Jun 20, 2026 at 2:54 PM Fujii Masao <[email protected]> wrote:
>
> >
> > Looks great! Thanks for fixing this.
>
> Thanks for the review! I've pushed the patches.
>
You seem to have forgotten to update the following comment in
pg_get_sequence_data(): "Return all NULLs for missing sequences,
sequences for which we lack
privileges, other sessions' temporary sequences, ...". BTW, if we go
by the logic of your proposal, shouldn't one also distinguish other
cases as mentioned in the comment quoted by me?
Also, we might want to consider additional errhint as follows:
errhint("Grant UPDATE on the sequence to the subscription/sequence
owner on the subscriber.")
errhint("Grant SELECT on the sequence to the replication role on the
publisher.")
--
With Regards,
Amit Kapila.
