On Fri, Oct 16, 2020 at 09:03:52AM +0000, Hou, Zhijie wrote:
> Hi
> 
> In /src/backend/executor/nodeAgg.c
> 
> I found the following comment still use work mem,
> Since hash_mem has been introduced, Is it more accurate to use hash_mem here ?
> 
> @@ -1827,7 +1827,7 @@ hash_agg_set_limits(double hashentrysize, double 
> input_groups, int used_bits,
>       /*
>        * Don't set the limit below 3/4 of hash_mem. In that case, we are at 
> the
>        * minimum number of partitions, so we aren't going to dramatically 
> exceed
> -      * work mem anyway.
> +      * hash_mem anyway.

Can someone comment on this?  Is the text change correct?

-- 
  Bruce Momjian  <br...@momjian.us>        https://momjian.us
  EDB                                      https://enterprisedb.com

  Only you can decide what is important to you.


Reply via email to