Re: Incorrect command tag row count for MERGE with a cross-partition update

2023-02-22 Thread Dean Rasheed
On Tue, 21 Feb 2023 at 09:34, Alvaro Herrera wrote: > > > I think the best fix is to have ExecMergeMatched() pass canSetTag = > > false to ExecUpdateAct(), so that ExecMergeMatched() takes > > responsibility for updating estate->es_processed in all cases. > > Sounds sensible. > I decided it was

Re: Incorrect command tag row count for MERGE with a cross-partition update

2023-02-21 Thread Alvaro Herrera
On 2023-Feb-20, Dean Rasheed wrote: > Playing around with MERGE some more, I noticed that the command tag > row count is wrong if it does a cross-partition update: > > CREATE TABLE target (a int, b int) PARTITION BY LIST (b); > CREATE TABLE target_p1 PARTITION OF target FOR VALUES IN (1); >

Incorrect command tag row count for MERGE with a cross-partition update

2023-02-20 Thread Dean Rasheed
Playing around with MERGE some more, I noticed that the command tag row count is wrong if it does a cross-partition update: CREATE TABLE target (a int, b int) PARTITION BY LIST (b); CREATE TABLE target_p1 PARTITION OF target FOR VALUES IN (1); CREATE TABLE target_p2 PARTITION OF target FOR VALUES