Mention jsonlog in description of logging_collector in GUC table

michaelpq · michaelpq · commit 061994fb3148 · 2025-02-02T11:31:31.000+09:00
logging_collector was only mentioning stderr and csvlog, and forgot about jsonlog. Oversight in dc68668, that has added support for jsonlog in log_destination. While on it, the description in the GUC table is tweaked to be more consistent with the documentation and postgresql.conf.sample. Author: Umar Hayat Reviewed-by: Ashutosh Bapat, Tom Lane Discussion: https://postgr.es/m/CAD68Dp1K_vBYqBEukHw=1jF7e76t8aszGZTFL2ugi=H7r=a7MA@mail.gmail.com Backpatch-through: 13
diff --git a/src/backend/utils/misc/guc.c b/src/backend/utils/misc/guc.c
@@ -1815,7 +1815,7 @@ static struct config_bool ConfigureNamesBool[] =
 	},
 	{
 		{"logging_collector", PGC_POSTMASTER, LOGGING_WHERE,
-			gettext_noop("Start a subprocess to capture stderr output and/or csvlogs into log files."),
+			gettext_noop("Start a subprocess to capture stderr, csvlog and/or jsonlog into log files."),
 			NULL
 		},
 		&Logging_collector,

Original file line number	Diff line number	Diff line change
`@@ -1815,7 +1815,7 @@ static struct config_bool ConfigureNamesBool[] =`
`1815`	`1815`	`},`
`1816`	`1816`	`{`
`1817`	`1817`	`{"logging_collector", PGC_POSTMASTER, LOGGING_WHERE,`
`1818`		`- gettext_noop("Start a subprocess to capture stderr output and/or csvlogs into log files."),`
	`1818`	`+ gettext_noop("Start a subprocess to capture stderr, csvlog and/or jsonlog into log files."),`
`1819`	`1819`	`NULL`
`1820`	`1820`	`},`
`1821`	`1821`	`&Logging_collector,`