Re: significant slowdown of HashAggregate between 9.6 and 10

2020-06-05 Thread Andres Freund
Hi, On 2020-06-05 15:25:26 +0200, Tomas Vondra wrote: > I think you're right. I think I was worried about having to resize the > hash table in case of an under-estimate, and it seemed fine to waste a > tiny bit more memory to prevent that. It's pretty cheap to resize a hashtable with a handful of

Re: significant slowdown of HashAggregate between 9.6 and 10

2020-06-05 Thread Tomas Vondra
On Thu, Jun 04, 2020 at 06:57:58PM -0700, Andres Freund wrote: Hi, On 2020-06-04 18:22:03 -0700, Jeff Davis wrote: On Thu, 2020-06-04 at 11:41 -0700, Andres Freund wrote: > +/* minimum number of initial hash table buckets */ > +#define HASHAGG_MIN_BUCKETS 256 > > > I don't really see much expla

Re: significant slowdown of HashAggregate between 9.6 and 10

2020-06-04 Thread Andres Freund
Hi, On 2020-06-04 18:22:03 -0700, Jeff Davis wrote: > On Thu, 2020-06-04 at 11:41 -0700, Andres Freund wrote: > > +/* minimum number of initial hash table buckets */ > > +#define HASHAGG_MIN_BUCKETS 256 > > > > > > I don't really see much explanation for that part in the commit, > > perhaps > >

Re: significant slowdown of HashAggregate between 9.6 and 10

2020-06-04 Thread Jeff Davis
On Thu, 2020-06-04 at 11:41 -0700, Andres Freund wrote: > +/* minimum number of initial hash table buckets */ > +#define HASHAGG_MIN_BUCKETS 256 > > > I don't really see much explanation for that part in the commit, > perhaps > Jeff can chime in? I did this in response to a review comment (point

Re: significant slowdown of HashAggregate between 9.6 and 10

2020-06-04 Thread Andres Freund
Hi, On 2020-06-03 13:26:43 -0700, Andres Freund wrote: > On 2020-06-03 21:31:01 +0200, Tomas Vondra wrote: > > So there seems to be +40% between 9.6 and 10, and further +25% between > > 10 and master. However, plain hashagg, measured e.g. like this: As far as I can tell the 10->master difference

Re: significant slowdown of HashAggregate between 9.6 and 10

2020-06-03 Thread Andres Freund
Hi, On 2020-06-03 21:31:01 +0200, Tomas Vondra wrote: > So there seems to be +40% between 9.6 and 10, and further +25% between > 10 and master. However, plain hashagg, measured e.g. like this: Ugh. Since I am a likely culprit, I'm taking a look. > Not sure what to think about this. Seems slot_

Re: significant slowdown of HashAggregate between 9.6 and 10

2020-06-03 Thread Tomas Vondra
Hi, Not sure what's the root cause, but I can reproduce it. Timings for 9.6, 10 and master (all built from git with the same options) without explain analyze look like this: 9.6 - Time: 1971.314 ms Time: 1995.875 ms Time: 1997.408 ms Time: 2069.913 ms Time: 2004.196 ms 10 --

Re: significant slowdown of HashAggregate between 9.6 and 10

2020-06-03 Thread Pavel Stehule
st 3. 6. 2020 v 17:32 odesílatel Pavel Stehule napsal: > Hi > > One czech Postgres user reported performance issue related to speed > HashAggregate in nested loop. > > The speed of 9.6 > > HashAggregate (cost=27586.10..27728.66 rows=14256 width=24) > (actual time=0.003..0.049 rows=39 loops=59920

significant slowdown of HashAggregate between 9.6 and 10

2020-06-03 Thread Pavel Stehule
Hi One czech Postgres user reported performance issue related to speed HashAggregate in nested loop. The speed of 9.6 HashAggregate (cost=27586.10..27728.66 rows=14256 width=24) (actual time=0.003..0.049 rows=39 loops=599203) The speed of 10.7 HashAggregate (cost=27336.78..27552.78 rows=2160