On Wed, Apr 30, 2025 at 09:57:46AM +0900, Michael Paquier wrote:
> On Tue, Apr 29, 2025 at 02:01:54PM -0400, Tom Lane wrote:
>> I'm inclined to argue that it's a bug fix and therefore still in-scope
>> for v18. The fact that we can't back-patch such a change is all the
>> more reason to not let it slide another year.
>
> Not on the RMT. I have looked at the patch, and I would agree with
> doing that now in v18 rather than wait for v19 to open up.
>
>> But probably the RMT should make the call.
>
> Yes.
I brought this up with the RMT, and everyone seemed okay with committing it
for v18.
> This has been mentioned upthread, but I am questioning the wisdom of
> putting the restriction based on MAX_RONAME_LEN only in
> pg_replication_origin_create(), while replorigin_create() is the one
> that actually matters. While it is true that these restrictions are
> enforced for the current callers in core, it could also be called by
> stuff outside of core. It seems important to me to let these
> potential callers know about the restriction in place.
I can move it back to replorigin_create(). I don't have a strong opinion
here.
--
nathan