On 11/5/21 08:55, Zhihong Yu wrote:
> + /* Check to see if subplan can be executed asynchronously */
> + if (subplan->async_capable)
> + {
> + subplan->async_capable = false;
>
> It seems the if statement is not needed: you can directly assign false
> to subplan->async_capable.Thank you, I agree with you.
Close look into the postgres_fdw regression tests show at least one open
problem with this approach: we need to control situations when only one
partition doesn't pruned and append isn't exist at all.
--
regards,
Andrey Lepikhov
Postgres Professional