Re: Make name optional in CREATE STATISTICS - Mailing list pgsql-hackers

From Simon Riggs
Subject Re: Make name optional in CREATE STATISTICS
Date
Msg-id CANbhV-HRr3nTNd1M_Tro5vXcdf1OSWTZVwgJO2Grr3S0bTmmGA@mail.gmail.com
Whole thread Raw
In response to Re: Make name optional in CREATE STATISTICS  (Matthias van de Meent <boekewurm+postgres@gmail.com>)
Responses Re: Make name optional in CREATE STATISTICS
List pgsql-hackers
On Thu, 7 Jul 2022 at 11:58, Matthias van de Meent
<boekewurm+postgres@gmail.com> wrote:
>
> On Thu, 7 Jul 2022 at 12:55, Simon Riggs <simon.riggs@enterprisedb.com> wrote:
> > There are various other ways of doing this and, yes, we could refactor
> > other parts of the grammar to make this work. There is a specific
> > guideline about patch submission that says the best way to get a patch
> > rejected is to include unnecessary changes. With that in mind, let's
> > keep the patch simple and exactly aimed at the original purpose.
> >
> > I'll leave it for committers to decide whether other refactoring is wanted.
>
> Fair enough.
>
> > I have made the comment show that the name is optional, thank you.
>
> The updated comment implies that IF NOT EXISTS is allowed without a
> defined name, which is false:
>
> > + *                CREATE STATISTICS [IF NOT EXISTS] [stats_name] [(stat types)]
>
> A more correct version would be
>
> + *                CREATE STATISTICS [ [IF NOT EXISTS] stats_name ]
> [(stat types)]

There you go

-- 
Simon Riggs                http://www.EnterpriseDB.com/

Attachment

pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: Non-replayable WAL records through overflows and >MaxAllocSize lengths
Next
From: Peter Eisentraut
Date:
Subject: Re: [RFC] building postgres with meson -v9