On Thu, Jan 21, 2021 at 1:50 PM Zhihong Yu <z...@yugabyte.com> wrote:
>
> For v12-0003-Enable-parallel-INSERT-and-or-SELECT-for-INSERT-INTO.patch:
>
> +       bool        isParallelModifyLeader = IsA(planstate, GatherState) && 
> IsA(outerPlanState(planstate), ModifyTableState);
>
> Please wrap long line.
>

OK.
I thought I ran pg_indent fairly recently, but maybe it chose not to
wrap that line.


> +   uint64     *processed_count_space;
>
> If I read the code correctly, it seems it can be dropped (use 
> pei->processed_count directly).
>

You're right. I'll change it in the next version.

Regards,
Greg Nancarrow
Fujitsu Australia


Reply via email to