On Mon, Feb 8, 2021 at 6:00 PM Hou, Zhijie <houzj.fnst@cn.fujitsu.com> wrote:
>
> > Posting an updated set of patches.
>
> A minor comment about doc.
>
> + <para>
> + Where the above target table features are determined to be, at worst,
> + parallel-restricted, rather than parallel-unsafe, at least a parallel table
> + scan may be used in the query plan for the <literal>INSERT</literal>
> + statement. For more information about Parallel Safety, see
> + <xref linkend="parallel-safety"/>.
> + </para>
>
> It seems does not mention that if target table is a foreign/temp table, a parallel table scan may be used.
>
> So how about:
>
> + <para>
> + Where the target table is a foreign/temporary table or the above target table features
> + are determined to be, at worst, parallel-restricted, rather than parallel-unsafe,
> + at least a parallel table scan may be used in the query plan for the
> + <literal>INSERT</literal> statement. For more information about Parallel Safety,
> + see <xref linkend="parallel-safety"/>.
> + </para>
>
Thanks. You're right, I should probably update the docs to clarify
those two cases.
(I had removed them from the list of parallel-unsafe things, but not
pointed out that a parallel table scan could still be used in these
cases).
Regards,
Greg Nancarrow
Fujitsu Australia
