> On Jul 26, 2026, at 03:30, Paul A Jungwirth <[email protected]> 
> wrote:
> 
> On Sat, Jul 25, 2026 at 3:16 AM Dean Rasheed <[email protected]> wrote:
>> 
>> The fix looks good to me, and the point about cross-partition updates
>> is a good one.
>> 
>> Attached is a v3 patch, in which I reworked the comments a bit, and
>> moved the logic of the new check to the place it's actually used. I
>> also changed the tests to use a NOTICE-raising function instead of a
>> sequence, since that makes it clearer which rows are/were being
>> processed by the RETURNING clause.
> 
> This all seems good. Using NOTICE does seem a little nicer.
> 
> The commit message has a small typo:
> 
>    . . . so that the auxiliary INSERT of a
>    cross-partition UPDATE with a FOR PORTION OF clause still process
> <-- s/b processes
>    RETURNING normally.

+1

Otherwise, v2 looks good to me.

Best regards,
--
Chao Li (Evan)
HighGo Software Co., Ltd.
https://www.highgo.com/






Reply via email to