Re: [HACKERS] [FEATURE PATCH] pg_stat_statements with plans (v02)

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: [HACKERS] [FEATURE PATCH] pg_stat_statements with plans (v02)
Дата
Msg-id 20369.1520020528@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: [HACKERS] [FEATURE PATCH] pg_stat_statements with plans (v02)  (Andres Freund <andres@anarazel.de>)
Список pgsql-hackers
Andres Freund <andres@anarazel.de> writes:
> Yea, I misread the diff to think you added a conflicting version. Due
> to:
> -DATA =3D pg_stat_statements--1.4.sql pg_stat_statements--1.4--1.5.sql \
> +DATA =3D pg_stat_statements--1.5.sql pg_stat_statements--1.4--1.5.sql \

> and I'd checked that 1.5 already exists. But you just renamed the file,
> presumably because it's essentially rewriting the whole file?  I'm not
> sure I'm a big fan of doing so, because that makes testing the upgrade
> path more work.

Yeah, that is not project style anymore.  Just add a delta file and leave
it at that.  See e.g. de1d042f5979bc1388e9a6d52a4d445342b04932 for an
example.

(We might from time to time roll up the deltas and replace the base
file with a newer version, but I think that should happen in separate
housekeeping commits.)

            regards, tom lane


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

Предыдущее
От: Andres Freund
Дата:
Сообщение: Re: 2018-03 Commitfest Summary (Andres #1)
Следующее
От: Julian Markwort
Дата:
Сообщение: Re: [HACKERS] [FEATURE PATCH] pg_stat_statements with plans (v02)