On Mon, Oct 30, 2017 at 1:49 PM, Thomas Munro <thomas.mu...@enterprisedb.com> wrote: > A couple of stupid things outstanding: > > 1. EXPLAIN ANALYZE for Parallel Hash "actual" shows the complete row > count, which is interesting to know (or not? maybe I should show it > somewhere else?), but the costing shows the partial row estimate used > for costing purposes.
Fixed. > 2. The BufFileSet's temporary directory gets created even if you > don't need it for batches. Duh. Fixed. I also refactored shared temporary files a bit while looking into this. The shared file ownership mechanism is now promoted to its own translation unit sharedfileset.c and it now works with fd.c files. buffile.c can still make use of it. That seems like a better division of labour. > 3. I don't have a good query rescan regression query yet. I wish I > could write my own query plans to test the executor. I found a query that rescans a parallel-aware hash join and added a couple of variants to the regression tests. I also decluttered the EXPLAIN ANALYZE output for enable_parallel_hash = off a bit. -- Thomas Munro http://www.enterprisedb.com
parallel-hash-v24.patchset.tgz
Description: GNU Zip compressed data
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
