On 2024-12-04 21:08, Yugo Nagata wrote:
This would prevent other users from make a misunderstanding as you did.
I found the description on amvalidate also explains a message is
output at INFO level.
( https://www.postgresql.org/docs/devel/index-functions.html )
Problems should be reported with ereport messages, typically at INFO
level.
I don't know the actual intention to mention the message level here,
but
this might have the same purpose of you.
Thanks for your response!
Should we add the description for CLUSTER command, since
it also has the VERBOSE options?
Yes, you're right. I missed it.
- Enables display of progress messages.
+ Prints a detailed analyze activity report for each table as
+ <literal>INFO</literal> messages.
Do we need rewrite the description in addition to mentioning the
message level?
Since the messages for the VERBOSE options look similar, I unified the
descriptions
for VACUUM, ANALYZE, and REINDEX. However, I don’t have a strong opinion
on this.
I’ve fixed it in the v2 patch.
Regards,
--
Masahiro Ikeda
NTT DATA CORPORATION
From f45982c314bda79904cbdcfee962040a1d30ae7e Mon Sep 17 00:00:00 2001
From: Masahiro Ikeda <ikeda...@oss.nttdata.com>
Date: Thu, 5 Dec 2024 10:00:16 +0900
Subject: [PATCH v2] Doc: clarify the log message level of the VERBOSE option
---
doc/src/sgml/ref/analyze.sgml | 2 +-
doc/src/sgml/ref/cluster.sgml | 3 ++-
doc/src/sgml/ref/reindex.sgml | 3 ++-
doc/src/sgml/ref/vacuum.sgml | 3 ++-
4 files changed, 7 insertions(+), 4 deletions(-)
diff --git a/doc/src/sgml/ref/analyze.sgml b/doc/src/sgml/ref/analyze.sgml
index a0db56ae74..ec81f00fec 100644
--- a/doc/src/sgml/ref/analyze.sgml
+++ b/doc/src/sgml/ref/analyze.sgml
@@ -65,7 +65,7 @@ ANALYZE [ ( <replaceable class="parameter">option</replaceable> [, ...] ) ] [ <r
<term><literal>VERBOSE</literal></term>
<listitem>
<para>
- Enables display of progress messages.
+ Enables display of progress messages at <literal>INFO</literal> level.
</para>
</listitem>
</varlistentry>
diff --git a/doc/src/sgml/ref/cluster.sgml b/doc/src/sgml/ref/cluster.sgml
index c5760244e6..434e3973ee 100644
--- a/doc/src/sgml/ref/cluster.sgml
+++ b/doc/src/sgml/ref/cluster.sgml
@@ -106,7 +106,8 @@ CLUSTER [ ( <replaceable class="parameter">option</replaceable> [, ...] ) ] [ <r
<term><literal>VERBOSE</literal></term>
<listitem>
<para>
- Prints a progress report as each table is clustered.
+ Prints a progress report as each table is clustered,
+ at <literal>INFO</literal> level.
</para>
</listitem>
</varlistentry>
diff --git a/doc/src/sgml/ref/reindex.sgml b/doc/src/sgml/ref/reindex.sgml
index dcf70d14bc..60da4c38e1 100644
--- a/doc/src/sgml/ref/reindex.sgml
+++ b/doc/src/sgml/ref/reindex.sgml
@@ -194,7 +194,8 @@ REINDEX [ ( <replaceable class="parameter">option</replaceable> [, ...] ) ] { DA
<term><literal>VERBOSE</literal></term>
<listitem>
<para>
- Prints a progress report as each index is reindexed.
+ Prints a progress report as each index is reindexed,
+ at <literal>INFO</literal> level.
</para>
</listitem>
</varlistentry>
diff --git a/doc/src/sgml/ref/vacuum.sgml b/doc/src/sgml/ref/vacuum.sgml
index 9110938fab..fd4de921dd 100644
--- a/doc/src/sgml/ref/vacuum.sgml
+++ b/doc/src/sgml/ref/vacuum.sgml
@@ -129,7 +129,8 @@ VACUUM [ ( <replaceable class="parameter">option</replaceable> [, ...] ) ] [ <re
<term><literal>VERBOSE</literal></term>
<listitem>
<para>
- Prints a detailed vacuum activity report for each table.
+ Prints a detailed vacuum activity report for each table,
+ at <literal>INFO</literal> level.
</para>
</listitem>
</varlistentry>
--
2.34.1
