Re: Proposal : REINDEX xxx VERBOSE

Поиск
Список
Период
Сортировка
От Sawada Masahiko
Тема Re: Proposal : REINDEX xxx VERBOSE
Дата
Msg-id CAD21AoCP-u1Yfpo7QwnBtgjSzOOzhr-sDS_7zPJhA21OrOWfdA@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Proposal : REINDEX xxx VERBOSE  (Robert Haas <robertmhaas@gmail.com>)
Ответы Re: Proposal : REINDEX xxx VERBOSE  (Robert Haas <robertmhaas@gmail.com>)
Список pgsql-hackers
On Thu, May 14, 2015 at 4:53 AM, Robert Haas <robertmhaas@gmail.com> wrote:
> On Wed, May 13, 2015 at 2:10 PM, Alvaro Herrera
> <alvherre@2ndquadrant.com> wrote:
>> Uh, are we really using INFO to log this?  I thought that was a
>> discouraged level to use anymore.  Why not NOTICE?
>
> Well, as Masahiko-san points out, VACUUM uses INFO.  I can't see any
> good reason to make this different.
>
>> Also, when multiple tables are reindexed, do we emit lines for each
>> index, or only for each table?  If we're going to emit a line for each
>> index in the single-table mode, it seems more sensible to do the same
>> for the multi-table forms also.
>
> Hmm, yeah, I agree with that.  I thought the patch worked that way,
> but I see now that it doesn't.  I think that should be changed.
>

The v15 patch emits a line for each table when reindexing multiple
tables, and emits a line for each index when reindexing single table.
But v14 patch emits a line for each index, regardless of reindex target.
Should I change back to v14 patch?

Regards,

-------
Sawada Masahiko



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

Предыдущее
От: Bruce Momjian
Дата:
Сообщение: Re: pgAdmin4 Bug fix or my Fault ?
Следующее
От: Fabrízio de Royes Mello
Дата:
Сообщение: Re: Proposal : REINDEX xxx VERBOSE