At Tue, 25 Apr 2017 11:00:16 -0400, Robert Haas <robertmh...@gmail.com> wrote in <CA+TgmobU=f1jtgz+bl91d7se2+cbybmj+wmqdmtt0cyv2dv...@mail.gmail.com> > On Mon, Apr 24, 2017 at 3:55 AM, Kyotaro HORIGUCHI > <horiguchi.kyot...@lab.ntt.co.jp> wrote: > > While I looked into dynahash.c, I found that the following > > sentense became outdated by 5dfc198. The commit removed the only > > usage of hash_freeze(). > > > >> * NOTE: it is possible to use hash_seq_init/hash_seq_search without any > >> * worry about hash_seq_term cleanup, if the hashtable is first locked > >> against > >> * further insertions by calling hash_freeze. This is used by nodeAgg.c, > >> * wherein it is inconvenient to track whether a scan is still open, and > >> * there's no possibility of further insertions after readout has begun. > > > > The attached patch removes the sentence begins with "This is used > > by..". > > Committed.
Thanks. -- Kyotaro Horiguchi NTT Open Source Software Center -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
