On Thu, Feb 20, 2020 at 09:38:26PM +0900, Amit Langote wrote: > Fujita-san, > > On Thu, Feb 20, 2020 at 8:36 PM Etsuro Fujita <etsuro.fuj...@gmail.com> wrote: > > partition_bounds_copy() sets the hash_part and natts variable in each > > iteration of a loop to copy the datums in the datums array, which > > would not be efficient. Attached is small patch for avoiding that. > > That looks good to me.
Looks good to me too!