Re: Minor improvement to partition_bounds_copy()

2020-02-21 Thread Etsuro Fujita
On Thu, Feb 20, 2020 at 10:52 PM Julien Rouhaud wrote: > On Thu, Feb 20, 2020 at 09:38:26PM +0900, Amit Langote wrote: > > On Thu, Feb 20, 2020 at 8:36 PM Etsuro Fujita > > wrote: > > > partition_bounds_copy() sets the hash_part and natts variable in each > > > iteration of a loop to copy the da

Re: Minor improvement to partition_bounds_copy()

2020-02-20 Thread Julien Rouhaud
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 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 effi

Re: Minor improvement to partition_bounds_copy()

2020-02-20 Thread Amit Langote
Fujita-san, On Thu, Feb 20, 2020 at 8:36 PM Etsuro Fujita 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

Minor improvement to partition_bounds_copy()

2020-02-20 Thread Etsuro Fujita
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. Best regards, Etsuro Fujita partition_bounds_copy.patch Description: Binary data