Mention jsonlog in description of logging_collector in GUC table

michaelpq · michaelpq · commit d61b9662b09e · 2025-02-02T11:31:21.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_tables.c b/src/backend/utils/misc/guc_tables.c
@@ -1692,7 +1692,7 @@ 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
`@@ -1692,7 +1692,7 @@ struct config_bool ConfigureNamesBool[] =`
`1692`	`1692`	`},`
`1693`	`1693`	`{`
`1694`	`1694`	`{"logging_collector", PGC_POSTMASTER, LOGGING_WHERE,`
`1695`		`- gettext_noop("Start a subprocess to capture stderr output and/or csvlogs into log files."),`
	`1695`	`+ gettext_noop("Start a subprocess to capture stderr, csvlog and/or jsonlog into log files."),`
`1696`	`1696`	`NULL`
`1697`	`1697`	`},`
`1698`	`1698`	`&Logging_collector,`