Mention jsonlog in description of logging_collector in GUC table

michaelpq · michaelpq · commit d7260f92adea · 2025-02-02T11:31:32.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
@@ -1766,7 +1766,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
`@@ -1766,7 +1766,7 @@ static struct config_bool ConfigureNamesBool[] =`
`1766`	`1766`	`},`
`1767`	`1767`	`{`
`1768`	`1768`	`{"logging_collector", PGC_POSTMASTER, LOGGING_WHERE,`
`1769`		`- gettext_noop("Start a subprocess to capture stderr output and/or csvlogs into log files."),`
	`1769`	`+ gettext_noop("Start a subprocess to capture stderr, csvlog and/or jsonlog into log files."),`
`1770`	`1770`	`NULL`
`1771`	`1771`	`},`
`1772`	`1772`	`&Logging_collector,`