No, it was wrong. At Mon, 29 Aug 2016 17:08:36 +0900 (Tokyo Standard Time), Kyotaro HORIGUCHI <horiguchi.kyot...@lab.ntt.co.jp> wrote in <20160829.170836.161449399.horiguchi.kyot...@lab.ntt.co.jp> > Hello, > > I considered applying the async infrastructure onto nodeGather, > but since parallel workers hardly make Gather (or the leader) > wait, it's really useless at least for simple cases. Furthermore, > as several people may have said before, being defferent from > foreign scans, gather (or other kinds of parallel) nodes usually > have several workers and will have up to two digit nubmers at the > most even on so-called many-core boxes. I finally gave up > applying this to nodeGather.
I overlooked that local scan takes place instead of waiting workers to be ready. I will reconsider counting that.. > As the result, the attached patchset is functionally the same > with the last version but replace misused Assert with > AssertMacro. -- Kyotaro Horiguchi NTT Open Source Software Center -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
