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

From Dean Rasheed
Subject Re: Make name optional in CREATE STATISTICS
Date
Msg-id CAEZATCW993Lt_bsT90VFMpxF-0s6aKw_4TesB_F5GqM8e1oP9A@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
Re: Make name optional in CREATE STATISTICS
Re: Make name optional in CREATE STATISTICS
List pgsql-hackers
On Wed, 20 Jul 2022 at 12:01, Matthias van de Meent
<boekewurm+postgres@gmail.com> wrote:
>
> On Wed, 13 Jul 2022 at 08:07, Simon Riggs <simon.riggs@enterprisedb.com> wrote:
> >
> > > + *                CREATE STATISTICS [ [IF NOT EXISTS] stats_name ]
>
> I think this is ready for a committer, so I've marked it as such.
>

Picking this up...

I tend to agree with Matthias' earlier point about avoiding code
duplication in the grammar. Without going off and refactoring other
parts of the grammar not related to this patch, it's still a slightly
smaller, simpler change, and less code duplication, to do this using a
new opt_stats_name production in the grammar, as in the attached.

I also noticed a comment in CreateStatistics() that needed updating.

Barring any further comments, I'll push this shortly.

Regards,
Dean

Attachment

pgsql-hackers by date:

Previous
From: Aleksander Alekseev
Date:
Subject: Re: [PATCH v1] eliminate duplicate code in table.c
Next
From: Aleksander Alekseev
Date:
Subject: Re: [PATCH v1] eliminate duplicate code in table.c