Re: minor change for create_list_bounds() - Mailing list pgsql-hackers

From Nathan Bossart
Subject Re: minor change for create_list_bounds()
Date
Msg-id 20220629234101.GA364747@nathanxps13
Whole thread Raw
In response to minor change for create_list_bounds()  (Zhihong Yu <zyu@yugabyte.com>)
Responses Re: minor change for create_list_bounds()  (David Rowley <dgrowleyml@gmail.com>)
List pgsql-hackers
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_bound_accepts_nulls().

Given this change probably doesn't meaningfully impact performance or code
clarity, I'm personally -1 for this patch.  Is there another motivation
that I am missing?

-- 
Nathan Bossart
Amazon Web Services: https://aws.amazon.com



pgsql-hackers by date:

Previous
From: Nathan Bossart
Date:
Subject: Re: replacing role-level NOINHERIT with a grant-level option
Next
From: "Jonathan S. Katz"
Date:
Subject: Re: PostgreSQL 15 beta 2 release announcement draft