]> granicus.if.org Git - postgresql/commitdiff
Reduce log verbosity of startup/shutdown for launcher subprocesses.
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 10 Mar 2017 20:18:38 +0000 (15:18 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 10 Mar 2017 20:18:38 +0000 (15:18 -0500)
There's no really good reason why the autovacuum launcher and logical
replication launcher should announce themselves at startup and shutdown
by default.  Users don't care that those processes exist, and it's
inconsistent that those background processes announce themselves while
others don't.  So, reduce those messages from LOG to DEBUG1 level.

I was sorely tempted to reduce the "starting logical replication worker
for subscription ..." message to DEBUG1 as well, but forebore for now.
Those processes might possibly be of direct interest to users, at least
until logical replication is a lot better shaken out than it is today.

Discussion: https://postgr.es/m/19479.1489121003@sss.pgh.pa.us

src/backend/postmaster/autovacuum.c
src/backend/replication/logical/launcher.c

index e8de9a3ced48816ad6f5b84d389f1317807be5a5..fa8de1390e56d65266c2118dcabf5232880dab7a 100644 (file)
@@ -412,7 +412,7 @@ AutoVacLauncherMain(int argc, char *argv[])
        /* Identify myself via ps */
        init_ps_display("autovacuum launcher process", "", "", "");
 
-       ereport(LOG,
+       ereport(DEBUG1,
                        (errmsg("autovacuum launcher started")));
 
        if (PostAuthDelay)
@@ -776,7 +776,7 @@ AutoVacLauncherMain(int argc, char *argv[])
 
        /* Normal exit from the autovac launcher is here */
 shutdown:
-       ereport(LOG,
+       ereport(DEBUG1,
                        (errmsg("autovacuum launcher shutting down")));
        AutoVacuumShmem->av_launcherpid = 0;
 
index afdadc17d6d9dd36d409431b6654a40eb349ea82..20b43626ddd612478b08b845a9e50406c7397bbc 100644 (file)
@@ -552,7 +552,7 @@ ApplyLauncherWakeup(void)
 void
 ApplyLauncherMain(Datum main_arg)
 {
-       ereport(LOG,
+       ereport(DEBUG1,
                        (errmsg("logical replication launcher started")));
 
        /* Establish signal handlers. */
@@ -652,7 +652,7 @@ ApplyLauncherMain(Datum main_arg)
        LogicalRepCtx->launcher_pid = 0;
 
        /* ... and if it returns, we're done */
-       ereport(LOG,
+       ereport(DEBUG1,
                        (errmsg("logical replication launcher shutting down")));
 
        proc_exit(0);