From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com> |
Cc: | Andres Freund <andres(at)anarazel(dot)de>, pgsql-committers <pgsql-committers(at)postgresql(dot)org> |
Subject: | Re: pgsql: Add parallel-aware hash joins. |
Date: | 2017-12-28 04:26:35 |
Message-ID: | 4503.1514435195@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com> writes:
> On Thu, Dec 28, 2017 at 3:32 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> Aside from the instability problems, I'm pretty unhappy about how much
>> the PHJ patch has added to the runtime of "make check". I do not think
>> any one feature can justify adding 20% to that. Can't you cut down the
>> amount of data processed by these new test cases?
> Isn't that mostly because of the CV livelock problem?
Hm, maybe. I quoted the 20% figure on the basis of longfin's reports,
not prairiedog's ... but it might be seeing some of the livelock effect
too.
> So without the effects of that bug it's only taking 2.4% longer than
> commit fa330f9a. Is that acceptable for a feature of this size and
> complexity? I will also look into making the data sets smaller.
That sounds better, but it's still worth asking whether the tests
could be quicker.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2017-12-28 11:14:11 | pgsql: Fix rare assertion failure in parallel hash join. |
Previous Message | Thomas Munro | 2017-12-28 04:15:00 | Re: pgsql: Add parallel-aware hash joins. |
From | Date | Subject | |
---|---|---|---|
Next Message | David Rowley | 2017-12-28 05:31:00 | Re: [HACKERS] Removing [Merge]Append nodes which contain a single subpath |
Previous Message | Thomas Munro | 2017-12-28 04:15:00 | Re: pgsql: Add parallel-aware hash joins. |