Mention jsonlog in description of logging_collector in GUC table

michaelpq · michaelpq · commit f07ebd54a304 · 2025-02-02T11:31:29.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
@@ -1630,7 +1630,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
`@@ -1630,7 +1630,7 @@ struct config_bool ConfigureNamesBool[] =`
`1630`	`1630`	`},`
`1631`	`1631`	`{`
`1632`	`1632`	`{"logging_collector", PGC_POSTMASTER, LOGGING_WHERE,`
`1633`		`- gettext_noop("Start a subprocess to capture stderr output and/or csvlogs into log files."),`
	`1633`	`+ gettext_noop("Start a subprocess to capture stderr, csvlog and/or jsonlog into log files."),`
`1634`	`1634`	`NULL`
`1635`	`1635`	`},`
`1636`	`1636`	`&Logging_collector,`