On Tue, 17 Aug 2021 at 19:06, 李杰(慎追) <adger.lj@alibaba-inc.com> wrote:
> Your answer explains that we still need to ModifyTable node without Leaf partitions.
> You are right about this.
>
> But you can review the source code again,
I'd been looking at the code in ExecInitModifyTable() that's the same
as what you pasted thinking you meant that. I think for the check for
partitioned tables in ExecModifyTable() then it's likely just dead
code. It seems to be causing a bit of confusion though, so might be
worth doing something about. Copied in Tom to see what he thinks as
it's one of his.
David
