Mention jsonlog in description of logging_collector in GUC table

michaelpq · michaelpq · commit 21d7db2caf2a · 2025-02-02T11:31:34.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
@@ -1696,7 +1696,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
`@@ -1696,7 +1696,7 @@ static struct config_bool ConfigureNamesBool[] =`
`1696`	`1696`	`},`
`1697`	`1697`	`{`
`1698`	`1698`	`{"logging_collector", PGC_POSTMASTER, LOGGING_WHERE,`
`1699`		`- gettext_noop("Start a subprocess to capture stderr output and/or csvlogs into log files."),`
	`1699`	`+ gettext_noop("Start a subprocess to capture stderr, csvlog and/or jsonlog into log files."),`
`1700`	`1700`	`NULL`
`1701`	`1701`	`},`
`1702`	`1702`	`&Logging_collector,`