On Fri, Mar 2, 2018 at 9:06 PM, Antonin Houska <a...@cybertec.at> wrote: > David Steele <da...@pgmasters.net> wrote: >> Does this look right to you? > > Yes, this is what I meant. The patch applies cleanly and the code compiles > well. > >> If so, can you sign up as a reviewer and mark it Ready for Committer? > > Done.
Thanks. > Actually I think it'd be nice if the "parallel_hash" argument was mentioned in > the header comment of initial_cost_hashjoin() function, but not sure this is > worth returning the patch to the author. Done. -- Thomas Munro http://www.enterprisedb.com
0001-Fix-costing-of-parallel-hash-joins-v2.patch
Description: Binary data
