Re: Emit namespace in post-copy output
От | Mike |
---|---|
Тема | Re: Emit namespace in post-copy output |
Дата | |
Msg-id | CAMerE0rgBxn+KvmqQdO2Oju5rjE43=Afi90GcpNGZCd6HyVDNA@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: Emit namespace in post-copy output (Corey Huinker <corey.huinker@gmail.com>) |
Ответы |
Re: Emit namespace in post-copy output
|
Список | pgsql-hackers |
Awesome, thanks! Are there any other steps I should take?
On Wed, Jun 23, 2021 at 5:46 PM Corey Huinker <corey.huinker@gmail.com> wrote:
On Tue, Jun 22, 2021 at 6:08 PM Mike <miketheman@gmail.com> wrote:When running a VACUUM or CLUSTER command, the namespace name is not part of the emitted message.Using `vacuumdb` CLI tool recently with multiple jobs, I found that reading the output messages harder to match the relations with their namespaces.Example:INFO: vacuuming "sendgrid.open"
INFO: vacuuming "mailgun.open"
...INFO: "open": found 0 removable, 31460776 nonremovable row versions in 1358656 pages
DETAIL: 0 dead row versions cannot be removed yet.
CPU 31.35s/261.26u sec elapsed 1620.68 sec....In this example. the user can't readily tell which `open` relation was completed.Attached is a patch using existing functions to include the namespace in the output string.Looking forward to feedback!-Mike FiedlerI've added this to the open commitfest: https://commitfest.postgresql.org/33/3200/The change is quite simple, just 3 lines, adding the schema name to two different lines of output.As such, there is no obvious documentation to change, though I can imagine that we have sample output from vacuum, vacuumdb or cluster somewhere that would need to be updated.
I cobbled together a very simple test:~/pgdata$ /usr/local/pgsql/bin/psql postgrespsql (14beta2)Type "help" for help.postgres=# create database mike_test;CREATE DATABASEpostgres=# \c mike_testYou are now connected to database "mike_test" as user "corey".mike_test=# create schema foo;CREATE SCHEMAmike_test=# create table foo.bar(x integer);CREATE TABLEmike_test=# \qmike_test=# VACUUM FULL VERBOSE foo.bar;INFO: vacuuming "foo.bar"INFO: "foo.bar": found 0 removable, 0 nonremovable row versions in 0 pagesDETAIL: 0 dead row versions cannot be removed yet.CPU: user: 0.00 s, system: 0.00 s, elapsed: 0.00 s.VACUUM
And of course vacuumdb~/pgdata$ /usr/local/pgsql/bin/vacuumdb --full --verbose mike_test --table=foo.barvacuumdb: vacuuming database "mike_test"INFO: vacuuming "foo.bar"INFO: "foo.bar": found 0 removable, 0 nonremovable row versions in 0 pagesDETAIL: 0 dead row versions cannot be removed yet.CPU: user: 0.00 s, system: 0.00 s, elapsed: 0.00 s.So far, so good.
В списке pgsql-hackers по дате отправления: