On Wed, Feb 5, 2020 at 4:49 AM Antonin Houska <a...@cybertec.at> wrote: > I can't figure out why ExecGather/ExecGatherMerge do check whether num_workers > is non-zero. I think the code would be a bit clearer if these tests were > replaced with Assert() statements, as the attached patch does.
Hmm. There are some cases where we plan on using a Gather node but then can't actually fire up parallelism because we run out of DSM segments or we run out of background workers. But the Gather is just part of the plan, so it would still have num_workers > 0 in those cases. This might just have been a thinko on my part, but I'm not totally sure. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company