Hi,
On 03/28/2017 11:07 AM, Rushabh Lathia wrote:
...
I think we all agree that we should get rid of nreaders from the
GatherMergeState and need to do some code re-factor. But if I
understood correctly that Robert's concern was to do that re-factor
as separate commit.
Maybe. It depends on how valuable it's to keep Gather and GatherMerge
similar - having nreaders in one and not the other seems a bit weird.
But maybe the refactoring would remove it from both nodes?
Also, it does not really solve the issue that we're using 'nreaders' or
'nworkers_launched' to access array with one extra element.
regards
--
Tomas Vondra http://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
