On Fri, 28 Mar 2025 at 16:40, Tender Wang <tndrwang@gmail.com> wrote: > > I tried a quick fix by adding "case CMD_NOTHING" after "case CMD_DELETE" like below: > > case CMD_DELETE: > case CMD_NOTHING: > break; > > It can work, and all regression tests pass. But I don't look into more details. This solution may not fix the root cause.
Yes, that looks like the right fix.
This appears to be a trivial oversight in ExecInitPartitionInfo(). I did a quick scan of other places that look at action->commandType and I didn't see any others that overlook CMD_NOTHING.
Yes, indeed. I double-check it.
In ExecInitPartitionInfo(), it has comments like:
* This duplicates much of the logic in ExecInitMerge(), so something * changes there, look here too.
