On Wed, 5 Aug 2020 at 14:27, David Rowley <dgrowle...@gmail.com> wrote: > > On Wed, 5 Aug 2020 at 14:13, Justin Pryzby <pry...@telsasoft.com> wrote: > > Also odd (to me). If I encourage more workers, there are "slots" for each > > "planned" worker, even though fewer were launched: > > Looking at explain.c for "num_workers; " (including the final space at > the end), looking at each for loop that loops over each worker, quite > a number of those locations have a condition that skips the worker. > > For example, show_sort_info() does > > if (sinstrument->sortMethod == SORT_TYPE_STILL_IN_PROGRESS) > continue; /* ignore any unfilled slots */ > > So maybe Hash Agg should be doing something similar. Additionally, > maybe it should not show the leader details if the leader didn't help.
Here's what I had in mind. The unpatched format got even more broken with EXPLAIN (ANALYZE, VERBOSE), so this is certainly a bug fix. David
dont_display_hashagg_properties_for_workers_that_dont_assist.patch
Description: Binary data
