Patch Review Complete: Multi-Batch Hash Join Improvements

From: Joshua Tolley <eggyknap(at)gmail(dot)com>
To: pgsql-hackers(at)postgresql(dot)org
Cc: ramon(dot)lawrence(at)ubc(dot)ca, pandasuit(at)gmail(dot)com
Subject: Patch Review Complete: Multi-Batch Hash Join Improvements
Date: 2008-11-18 06:19:51
Message-ID: 20081118061951.GA10099@polonium.part.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Note: this email is effectively a repeat of an email sent earlier to
which there has been less response than I expected. If there's something
else I'm supposed to do at this point, someone please let me know,
because I don't know what it is :)

-----------------------------------

I've finished all I think I can do to review the multi-batch hash join
improvement patch (http://is.gd/7W8Z). It appears to work as advertised:

-- it speeds up joins by fairly significant margins in some cases
-- I've been unable to make a case where it performs worse than the
existing implementation despite deliberately pathological inputs
-- it applies cleanly
-- it doesn't break stuff, AFAICS

I don't consider myself qualified to do a proper code review. I can only
suggest the author modify it to use /* ... */ style comments rather than
//... comments.

If there's more I should do, someone please let me know. Otherwise, I
leave it in the capable hands of the InterWebs.

- Josh / eggyknap

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Jeff Davis 2008-11-18 06:42:21 Re: Patch Review Complete: Multi-Batch Hash Join Improvements
Previous Message KaiGai Kohei 2008-11-18 06:02:39 Re: Updates of SE-PostgreSQL 8.4devel patches (r1197)