Re: Make name optional in CREATE STATISTICS

Поиск
Список
Период
Сортировка
От Dean Rasheed
Тема Re: Make name optional in CREATE STATISTICS
Дата
Msg-id CAEZATCW993Lt_bsT90VFMpxF-0s6aKw_4TesB_F5GqM8e1oP9A@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Make name optional in CREATE STATISTICS  (Matthias van de Meent <boekewurm+postgres@gmail.com>)
Ответы Re: Make name optional in CREATE STATISTICS  (Simon Riggs <simon.riggs@enterprisedb.com>)
Re: Make name optional in CREATE STATISTICS  (Tomas Vondra <tomas.vondra@enterprisedb.com>)
Re: Make name optional in CREATE STATISTICS  (Alvaro Herrera <alvherre@alvh.no-ip.org>)
Список 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

Вложения

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Aleksander Alekseev
Дата:
Сообщение: Re: [PATCH v1] eliminate duplicate code in table.c
Следующее
От: Aleksander Alekseev
Дата:
Сообщение: Re: [PATCH v1] eliminate duplicate code in table.c