Re: minor change for create_list_bounds()

2022-07-13 Thread Nathan Bossart
On Wed, Jul 13, 2022 at 05:07:53PM +1200, David Rowley wrote: > While I agree that the gains on making this change are small. It just > accounts to saving a call to bms_add_member() when we've already found > the partition to be interleaved due to interleaved Datum values, I > just disagree with n

Re: minor change for create_list_bounds()

2022-07-12 Thread David Rowley
On Thu, 30 Jun 2022 at 11:41, Nathan Bossart wrote: > > On Tue, Mar 08, 2022 at 11:05:10AM -0800, Zhihong Yu wrote: > > I was looking at commit db632fbca and noticed that, > > in create_list_bounds(), if index is added to boundinfo->interleaved_parts > > in the first if statement, there is no need

Re: minor change for create_list_bounds()

2022-06-29 Thread Nathan Bossart
On Tue, Mar 08, 2022 at 11:05:10AM -0800, Zhihong Yu wrote: > I was looking at commit db632fbca and noticed that, > in create_list_bounds(), if index is added to boundinfo->interleaved_parts > in the first if statement, there is no need to perform the second check > involving call to partition_boun