On Thu, 23 May 2024 at 04:26, David Rowley <dgrowle...@gmail.com> wrote: > > On Sun, 19 May 2024 at 15:20, David Rowley <dgrowle...@gmail.com> wrote: > > > > I noticed that PlannedStmt.hasReturning and hasModifyingCTE have an > > outdated comment now that MERGE supports RETURNING (per commit > > c649fa24a) > > > > i.e. these two: > > > > > bool hasReturning; /* is it insert|update|delete RETURNING? */ > > > > > bool hasModifyingCTE; /* has insert|update|delete in WITH? */ > > I've pushed the fix for that. >
Thanks for taking care of that. I found another couple of similar comments that also needed updating, so I've pushed a fix for them too. Regards, Dean