From 9807b9e4b2e78ae3b5a0dd5e6c28eb1ebbd6756d Mon Sep 17 00:00:00 2001 From: Fujii Masao Date: Thu, 19 Mar 2026 20:32:43 +0900 Subject: [PATCH v2 1/2] Reduce log level of some logical decoding messages from LOG to DEBUG1 Previously some logical decoding messages (e.g., "logical decoding found consistent point") were logged at level LOG, even though they provided low-level, developer-oriented information that DBAs were typically not interested in. Since these messages can occur routinely (for example, when keeping calling pg_logical_slot_get_changes() to obtain the changes from logical decoding), logging them at LOG can be overly verbose. This commit reduces their log level to DEBUG1 to avoid unnecessary log noise. --- src/backend/replication/logical/logical.c | 2 +- src/backend/replication/logical/snapbuild.c | 6 +++--- src/test/recovery/t/038_save_logical_slots_shutdown.pl | 1 + 3 files changed, 5 insertions(+), 4 deletions(-) diff --git a/src/backend/replication/logical/logical.c b/src/backend/replication/logical/logical.c index 603a2b94d05..f20d0c542f3 100644 --- a/src/backend/replication/logical/logical.c +++ b/src/backend/replication/logical/logical.c @@ -598,7 +598,7 @@ CreateDecodingContext(XLogRecPtr start_lsn, ctx->reorder->output_rewrites = ctx->options.receive_rewrites; - ereport(LOG, + ereport(DEBUG1, (errmsg("starting logical decoding for slot \"%s\"", NameStr(slot->data.name)), errdetail("Streaming transactions committing after %X/%08X, reading WAL from %X/%08X.", diff --git a/src/backend/replication/logical/snapbuild.c b/src/backend/replication/logical/snapbuild.c index 37f0c6028bd..b4269a3b102 100644 --- a/src/backend/replication/logical/snapbuild.c +++ b/src/backend/replication/logical/snapbuild.c @@ -1312,7 +1312,7 @@ SnapBuildFindSnapshot(SnapBuild *builder, XLogRecPtr lsn, xl_running_xacts *runn builder->state = SNAPBUILD_CONSISTENT; builder->next_phase_at = InvalidTransactionId; - ereport(LOG, + ereport(DEBUG1, errmsg("logical decoding found consistent point at %X/%08X", LSN_FORMAT_ARGS(lsn)), errdetail("There are no running transactions.")); @@ -1409,7 +1409,7 @@ SnapBuildFindSnapshot(SnapBuild *builder, XLogRecPtr lsn, xl_running_xacts *runn builder->state = SNAPBUILD_CONSISTENT; builder->next_phase_at = InvalidTransactionId; - ereport(LOG, + ereport(DEBUG1, errmsg("logical decoding found consistent point at %X/%08X", LSN_FORMAT_ARGS(lsn)), errdetail("There are no old transactions anymore.")); @@ -1915,7 +1915,7 @@ SnapBuildRestore(SnapBuild *builder, XLogRecPtr lsn) Assert(builder->state == SNAPBUILD_CONSISTENT); - ereport(LOG, + ereport(DEBUG1, errmsg("logical decoding found consistent point at %X/%08X", LSN_FORMAT_ARGS(lsn)), errdetail("Logical decoding will begin using saved snapshot.")); diff --git a/src/test/recovery/t/038_save_logical_slots_shutdown.pl b/src/test/recovery/t/038_save_logical_slots_shutdown.pl index c0392d50460..05aa78b4bc7 100644 --- a/src/test/recovery/t/038_save_logical_slots_shutdown.pl +++ b/src/test/recovery/t/038_save_logical_slots_shutdown.pl @@ -48,6 +48,7 @@ $node_publisher->append_conf( 'postgresql.conf', q{ checkpoint_timeout = 1h autovacuum = off +log_min_messages = 'debug1' }); $node_publisher->start; -- 2.51.2