From 4bc3d0dcfc3ccf6e90611e5f2d9f1b098e17f3fa Mon Sep 17 00:00:00 2001 From: Artur Zakirov Date: Fri, 13 Feb 2026 12:41:14 +0100 Subject: [PATCH] Issue #478: Replace client side LOG messages by INFO Client code shouldn't log LOG level messages since it is considered as a server operational messages according to PostgreSQL logic. The function `log_required()`, which is used by pg_repack, is basically copy of a similar function from PostgreSQL code, therefore we should follow same logic. --- bin/pg_repack.c | 6 +++--- bin/pgut/pgut.c | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/bin/pg_repack.c b/bin/pg_repack.c index 97335b5e..aa7476de 100644 --- a/bin/pg_repack.c +++ b/bin/pg_repack.c @@ -1059,7 +1059,7 @@ rebuild_indexes(const repack_table *table) /* Assign available worker to build an index. */ index_jobs[i].status = INPROGRESS; index_jobs[i].worker_idx = i; - elog(LOG, "Initial worker %d to build index: %s", + elog(INFO, "Initial worker %d to build index: %s", i, index_jobs[i].create_index); if (!(PQsendQuery(workers.conns[i], index_jobs[i].create_index))) @@ -1154,7 +1154,7 @@ rebuild_indexes(const repack_table *table) } if (!PQisBusy(workers.conns[index_jobs[i].worker_idx])) { - elog(LOG, "Command finished in worker %d: %s", + elog(INFO, "Command finished in worker %d: %s", index_jobs[i].worker_idx, index_jobs[i].create_index); @@ -1191,7 +1191,7 @@ rebuild_indexes(const repack_table *table) { index_jobs[i].status = INPROGRESS; index_jobs[i].worker_idx = freed_worker; - elog(LOG, "Assigning worker %d to build index #%d: " + elog(INFO, "Assigning worker %d to build index #%d: " "%s", freed_worker, i, index_jobs[i].create_index); diff --git a/bin/pgut/pgut.c b/bin/pgut/pgut.c index 106d8dda..6b004fe0 100644 --- a/bin/pgut/pgut.c +++ b/bin/pgut/pgut.c @@ -613,11 +613,11 @@ echo_query(const char *query, int nParams, const char **params) int i; if (strchr(query, '\n')) - elog(LOG, "(query)\n%s", query); + elog(INFO, "(query)\n%s", query); else - elog(LOG, "(query) %s", query); + elog(INFO, "(query) %s", query); for (i = 0; i < nParams; i++) - elog(LOG, "\t(param:%d) = %s", i, params[i] ? params[i] : "(null)"); + elog(INFO, "\t(param:%d) = %s", i, params[i] ? params[i] : "(null)"); } PGresult *