Hi, On Tue, Feb 10, 2026 at 09:35:26AM +0800, Chao Li wrote: > > > > On Feb 5, 2026, at 23:00, Ashutosh Bapat <[email protected]> > > wrote: > > > > On Thu, Feb 5, 2026 at 9:09 AM wangpeng <[email protected]> wrote: > >> > >> Hi hackers, > >> > >> This is my first patch > > > > Welcome to the community. > > > >> There is an incorrect assignment of nodeid = 0 in > >> TransactionIdGetCommitTsData() (commit_ts.c). > >> Elsewhere in the file, the code consistently uses InvalidReplOriginId. > >> Even though InvalidReplOriginId also evaluates to 0, it should be used > >> here for clarity and consistency. > > > > Few lines below we have a similar assignment which uses > > InvalidReplOriginId. I agree with your suggestion. Patch LGTM. It's an > > old and small commit so maybe Alvaro, who committed the change > > (4aaddf2f009821e29aea3735e44332ad9ca47aaa), may not remember it. Still > > including him in case. > > > > -- > > Best Wishes, > > Ashutosh Bapat > > > > > > while this patch just fixes one remaining omission.
I did check with the same kind of Coccinelle script used for ec317440716 and it looks like the proposed change here for InvalidReplOriginId is the only one to fix for the ReplOriginId type. FWIW, I extended the exercise for other Invalid* values in [1]. [1]: https://postgr.es/m/aY2oKlSpikgO9m%2BX%40ip-10-97-1-34.eu-west-3.compute.internal Regards, -- Bertrand Drouvot PostgreSQL Contributors Team RDS Open Source Databases Amazon Web Services: https://aws.amazon.com
