On Thu, 21 Jul 2022 at 15:12, Dean Rasheed <dean.a.rash...@gmail.com> wrote: > > On Wed, 20 Jul 2022 at 12:01, Matthias van de Meent > <boekewurm+postg...@gmail.com> wrote: > > > > On Wed, 13 Jul 2022 at 08:07, Simon Riggs <simon.ri...@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.
Nice change, please proceed. -- Simon Riggs http://www.EnterpriseDB.com/
