Re: pgsql: Add per-index stats information in verbose logs of autovacuum
От | Peter Geoghegan |
---|---|
Тема | Re: pgsql: Add per-index stats information in verbose logs of autovacuum |
Дата | |
Msg-id | CAH2-Wz=kF4vvniE_tNfKqar_+_GwdoHgK-Dma+d_EjuWDebQiQ@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: pgsql: Add per-index stats information in verbose logs of autovacuum (Michael Paquier <michael@paquier.xyz>) |
Ответы |
Re: pgsql: Add per-index stats information in verbose logs of autovacuum
|
Список | pgsql-committers |
On Mon, Mar 22, 2021 at 11:22 PM Michael Paquier <michael@paquier.xyz> wrote: > > To be clear, I agree that it would not be sensible to make the > > log_autovacuum_min_duration output 100% consistent with VACUUM > > VERBOSE. I just think that "%u remain" is misleading. It's just that > > one detail. > > Saying that, I like your suggestion of using "total" or "in total" > instead of "remain" for the first item. I'm okay with preserving "pages:" if you prefer it that way. It's really just the word "remain" -- it reminds me of the old "pages_removed" field. That used to be a thing that we showed back when old-style VACUUM existed and could shrink the index -- this is of course no longer possible. The message should clearly convey that the first number shown is simply the total size of the index. -- Peter Geoghegan
В списке pgsql-committers по дате отправления: