On Wed, May 25, 2022 at 1:04 PM Amit Langote <amitlangot...@gmail.com> wrote:
> On Wed, May 25, 2022 at 12:44 PM David Rowley <dgrowle...@gmail.com> wrote:
> > On Wed, 25 May 2022 at 15:01, Amit Langote <amitlangot...@gmail.com> wrote:
> > > +Previously, a partitioned table with DEFAULT partition or a LIST
> > > partition containing multiple values could not be used for ordered
> > > partition scans. Now it can be used at least in the cases where such
> > > partitions are pruned.
> >
> > I think this one is an improvement.  I'd drop "at least".
>
> Okay, I can agree that "at least" sounds a bit extraneous, so removed.

* I think it's better to s/...or a LIST partition/...or with a LIST partition

* The capitalization of DEFAULT and LIST seems unnecessary.

Updated the patch on those points.

-- 
Thanks, Amit Langote
EDB: http://www.enterprisedb.com

Attachment: reword-ordered-partition-scan-item_v5.diff
Description: Binary data

Reply via email to