On 12/21/2016 04:53 PM, Dilip Kumar wrote:
On Wed, Dec 21, 2016 at 8:39 PM, Robert Haas <robertmh...@gmail.com> wrote:
Committed the refactoring patch with some mild cosmetic adjustments.


As to the second patch:

+        if (jointype == JOIN_UNIQUE_INNER)
+            jointype = JOIN_INNER;

Isn't this dead code?  save_jointype might that value, but jointype won't.

Yes, it is.

I have fixed this, and also observed that comment for
try_partial_mergejoin_path header was having some problem, fixed that

FWIW, I've done quite a bit of testing on this patch, and also on the other patches adding parallel index scans and bitmap heap scan. I've been running TPC-H and TPC-DS on 16GB data sets with each patch, looking for regressions or crashes.

I haven't found any of that so far, which is good of course. It however seems the plan changes only for very few queries in those benchmarks with any of the patches, even after tweaking the costs to make parallel plans more likely.

I'm going to try with larger scales and also --enable-cassert and post the results during CF 2017-1.


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:

Reply via email to