Thread
Commits
-
doc: Add note about pg_settings and customized options into catalogs.sgml.
- ef60de67ebde 14.0 landed
-
document pg_settings view doesn't display custom options
John Naylor <john.naylor@enterprisedb.com> — 2020-10-28T18:15:27Z
Starting separate threads to keep from cluttering the TODO list thread. Here's a patch for the subject, as mentioned in https://www.postgresql.org/message-id/20201027220555.GS4951%40momjian.us -- John Naylor EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company
-
Re: document pg_settings view doesn't display custom options
John Naylor <john.naylor@enterprisedb.com> — 2020-10-28T18:45:30Z
On Wed, Oct 28, 2020 at 2:15 PM John Naylor <john.naylor@enterprisedb.com> wrote: > Starting separate threads to keep from cluttering the TODO list thread. > > Here's a patch for the subject, as mentioned in > https://www.postgresql.org/message-id/20201027220555.GS4951%40momjian.us > I just realized I introduced a typo, so here's v2. -- John Naylor EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company
-
Re: document pg_settings view doesn't display custom options
Fujii Masao <masao.fujii@oss.nttdata.com> — 2020-10-29T03:38:08Z
On 2020/10/29 3:45, John Naylor wrote: > On Wed, Oct 28, 2020 at 2:15 PM John Naylor <john.naylor@enterprisedb.com <mailto:john.naylor@enterprisedb.com>> wrote: > > Starting separate threads to keep from cluttering the TODO list thread. > > Here's a patch for the subject, as mentioned in > https://www.postgresql.org/message-id/20201027220555.GS4951%40momjian.us > > > I just realized I introduced a typo, so here's v2. + The <structname>pg_settings</structname> view does not display + <link linkend="runtime-config-custom">customized options</link>. This is true until the module that defines the customized options is loaded, but not after that. No? For example, pg_settings displays pg_stat_statements.max after pg_stat_statements is loaded. Regards, -- Fujii Masao Advanced Computing Technology Center Research and Development Headquarters NTT DATA CORPORATION
-
Re: document pg_settings view doesn't display custom options
John Naylor <john.naylor@enterprisedb.com> — 2020-10-29T12:54:07Z
On Wed, Oct 28, 2020 at 11:38 PM Fujii Masao <masao.fujii@oss.nttdata.com> wrote: > > > On 2020/10/29 3:45, John Naylor wrote: > > On Wed, Oct 28, 2020 at 2:15 PM John Naylor < > john.naylor@enterprisedb.com <mailto:john.naylor@enterprisedb.com>> wrote: > > > > Starting separate threads to keep from cluttering the TODO list > thread. > > > > Here's a patch for the subject, as mentioned in > > > https://www.postgresql.org/message-id/20201027220555.GS4951%40momjian.us > > > > > > I just realized I introduced a typo, so here's v2. > > + The <structname>pg_settings</structname> view does not display > + <link linkend="runtime-config-custom">customized options</link>. > > This is true until the module that defines the customized options is > loaded, > but not after that. No? For example, pg_settings displays > pg_stat_statements.max after pg_stat_statements is loaded. > True, how about this: The <structname>pg_settings</structname> does not display <link linkend="runtime-config-custom">customized options</link> that have been set before the relevant extension module has been loaded. -- John Naylor EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company
-
Re: document pg_settings view doesn't display custom options
Fujii Masao <masao.fujii@oss.nttdata.com> — 2020-10-30T03:50:59Z
On 2020/10/29 21:54, John Naylor wrote: > > > On Wed, Oct 28, 2020 at 11:38 PM Fujii Masao <masao.fujii@oss.nttdata.com <mailto:masao.fujii@oss.nttdata.com>> wrote: > > > > On 2020/10/29 3:45, John Naylor wrote: > > On Wed, Oct 28, 2020 at 2:15 PM John Naylor <john.naylor@enterprisedb.com <mailto:john.naylor@enterprisedb.com> <mailto:john.naylor@enterprisedb.com <mailto:john.naylor@enterprisedb.com>>> wrote: > > > > Starting separate threads to keep from cluttering the TODO list thread. > > > > Here's a patch for the subject, as mentioned in > > https://www.postgresql.org/message-id/20201027220555.GS4951%40momjian.us > > > > > > I just realized I introduced a typo, so here's v2. > > + The <structname>pg_settings</structname> view does not display > + <link linkend="runtime-config-custom">customized options</link>. > > This is true until the module that defines the customized options is loaded, > but not after that. No? For example, pg_settings displays > pg_stat_statements.max after pg_stat_statements is loaded. > > > True, how about this: > > The <structname>pg_settings</structname> does not display > <link linkend="runtime-config-custom">customized options</link> > that have been set before the relevant extension module has been loaded. I guess that someone can misread this as customized options that have been set before the relevant extension module has been loaded are not displayed even after the module is loaded. So what about the following, instead? The pg_settings does not display customized options until the extension module that defines them has been loaded. Also I think this note should be in the different paragraph from the paragraph of "The pg_settings view cannot be inserted into or deleted from...." because they are different topics. Thought? Regards, -- Fujii Masao Advanced Computing Technology Center Research and Development Headquarters NTT DATA CORPORATION -
Re: document pg_settings view doesn't display custom options
John Naylor <john.naylor@enterprisedb.com> — 2020-10-30T16:01:04Z
On Thu, Oct 29, 2020 at 11:51 PM Fujii Masao <masao.fujii@oss.nttdata.com> wrote: > > > On 2020/10/29 21:54, John Naylor wrote: > > > The <structname>pg_settings</structname> does not display > > <link linkend="runtime-config-custom">customized options</link> > > that have been set before the relevant extension module has been > loaded. > > I guess that someone can misread this as > > customized options that have been set before the relevant extension > module has been loaded are not displayed even after the module is > loaded. > > So what about the following, instead? > > The pg_settings does not display customized options until the > extension > module that defines them has been loaded. > > Also I think this note should be in the different paragraph from the > paragraph > of "The pg_settings view cannot be inserted into or deleted from...." > because > they are different topics. Thought? > Agreed on both points. In a separate paragraph, I think it's awkward to start two consecutive sentences with "The pg_settings view". If we put it in the previous paragraph we could phrase it like this: "See Section 20.1 for more information about the various ways to change these parameters. Customized options are not displayed until the extension module that defines them has been loaded. The pg_settings view..." -- John Naylor EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company
-
Re: document pg_settings view doesn't display custom options
Tom Lane <tgl@sss.pgh.pa.us> — 2020-10-30T16:07:11Z
John Naylor <john.naylor@enterprisedb.com> writes: > On Thu, Oct 29, 2020 at 11:51 PM Fujii Masao <masao.fujii@oss.nttdata.com> > wrote: >> Also I think this note should be in the different paragraph from the >> paragraph >> of "The pg_settings view cannot be inserted into or deleted from...." >> because >> they are different topics. Thought? > Agreed on both points. In a separate paragraph, I think it's awkward to > start two consecutive sentences with "The pg_settings view". If we put it > in the previous paragraph we could phrase it like this: > "See Section 20.1 for more information about the various ways to change > these parameters. Customized options are not displayed until the > extension module that defines them has been loaded. That just moves the subject-inconsistency to a different para :-( I think this item should be its own new para. As for the repetitiveness, we could just say "This view ...", in one or even both paras. regards, tom lane
-
Re: document pg_settings view doesn't display custom options
John Naylor <john.naylor@enterprisedb.com> — 2020-10-30T16:35:47Z
On Fri, Oct 30, 2020 at 12:07 PM Tom Lane <tgl@sss.pgh.pa.us> wrote: > John Naylor <john.naylor@enterprisedb.com> writes: > > On Thu, Oct 29, 2020 at 11:51 PM Fujii Masao < > masao.fujii@oss.nttdata.com> > > wrote: > >> Also I think this note should be in the different paragraph from the > >> paragraph > >> of "The pg_settings view cannot be inserted into or deleted from...." > >> because > >> they are different topics. Thought? > > > Agreed on both points. In a separate paragraph, I think it's awkward to > > start two consecutive sentences with "The pg_settings view". If we put it > > in the previous paragraph we could phrase it like this: > > > "See Section 20.1 for more information about the various ways to change > > these parameters. Customized options are not displayed until the > > extension module that defines them has been loaded. > > That just moves the subject-inconsistency to a different para :-( > I think this item should be its own new para. > > As for the repetitiveness, we could just say "This view ...", in one or > even both paras. > Okay, along those lines here's a patch using "this view" in a new paragraph for simplicity. -- John Naylor EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company
-
Re: document pg_settings view doesn't display custom options
Tom Lane <tgl@sss.pgh.pa.us> — 2020-10-30T16:48:24Z
John Naylor <john.naylor@enterprisedb.com> writes: > Okay, along those lines here's a patch using "this view" in a new paragraph > for simplicity. Basically OK with me, but ... <quibble> It seems fairly weird to use a nonspecific reference first and then a specific one. That is, I'd expect to read "The pg_settings view ..." and then "This view ...", not the other way around. So we could put this para second, or put it first but make this para say "The pg_settings view ..." while the existing text gets reduced to "This view ...". Or just make them both say "This view ..." so we don't have to have this discussion again the next time somebody wants to add a para here. </quibble> regards, tom lane
-
Re: document pg_settings view doesn't display custom options
John Naylor <john.naylor@enterprisedb.com> — 2020-10-30T17:06:56Z
On Fri, Oct 30, 2020 at 12:48 PM Tom Lane <tgl@sss.pgh.pa.us> wrote: > John Naylor <john.naylor@enterprisedb.com> writes: > > Okay, along those lines here's a patch using "this view" in a new > paragraph > > for simplicity. > > Basically OK with me, but ... > > <quibble> > It seems fairly weird to use a nonspecific reference first and then a > specific one. That is, I'd expect to read "The pg_settings view ..." > and then "This view ...", not the other way around. So we could > put this para second, or put it first but make this para say > "The pg_settings view ..." while the existing text gets reduced to > "This view ...". > > Or just make them both say "This view ..." so we don't have to have > this discussion again the next time somebody wants to add a para here. > </quibble> > Okay, how's this? -- John Naylor EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company
-
Re: document pg_settings view doesn't display custom options
Fujii Masao <masao.fujii@oss.nttdata.com> — 2020-11-06T15:42:34Z
On 2020/10/31 2:06, John Naylor wrote: > > > On Fri, Oct 30, 2020 at 12:48 PM Tom Lane <tgl@sss.pgh.pa.us <mailto:tgl@sss.pgh.pa.us>> wrote: > > John Naylor <john.naylor@enterprisedb.com <mailto:john.naylor@enterprisedb.com>> writes: > > Okay, along those lines here's a patch using "this view" in a new paragraph > > for simplicity. > > Basically OK with me, but ... > > <quibble> > It seems fairly weird to use a nonspecific reference first and then a > specific one. That is, I'd expect to read "The pg_settings view ..." > and then "This view ...", not the other way around. So we could > put this para second, or put it first but make this para say > "The pg_settings view ..." while the existing text gets reduced to > "This view ...". > > Or just make them both say "This view ..." so we don't have to have > this discussion again the next time somebody wants to add a para here. > </quibble> > > > Okay, how's this? Looks good to me. Barring any objection, I will commit the patch. Regards, -- Fujii Masao Advanced Computing Technology Center Research and Development Headquarters NTT DATA CORPORATION
-
Re: document pg_settings view doesn't display custom options
Fujii Masao <masao.fujii@oss.nttdata.com> — 2020-11-09T06:12:47Z
On 2020/11/07 0:42, Fujii Masao wrote: > > > On 2020/10/31 2:06, John Naylor wrote: >> >> >> On Fri, Oct 30, 2020 at 12:48 PM Tom Lane <tgl@sss.pgh.pa.us <mailto:tgl@sss.pgh.pa.us>> wrote: >> >> John Naylor <john.naylor@enterprisedb.com <mailto:john.naylor@enterprisedb.com>> writes: >> > Okay, along those lines here's a patch using "this view" in a new paragraph >> > for simplicity. >> >> Basically OK with me, but ... >> >> <quibble> >> It seems fairly weird to use a nonspecific reference first and then a >> specific one. That is, I'd expect to read "The pg_settings view ..." >> and then "This view ...", not the other way around. So we could >> put this para second, or put it first but make this para say >> "The pg_settings view ..." while the existing text gets reduced to >> "This view ...". >> >> Or just make them both say "This view ..." so we don't have to have >> this discussion again the next time somebody wants to add a para here. >> </quibble> >> >> >> Okay, how's this? > > Looks good to me. Barring any objection, I will commit the patch. Pushed. Thanks! Regards, -- Fujii Masao Advanced Computing Technology Center Research and Development Headquarters NTT DATA CORPORATION
-
Re: document pg_settings view doesn't display custom options
John Naylor <john.naylor@enterprisedb.com> — 2020-11-09T13:41:01Z
On Mon, Nov 9, 2020 at 2:12 AM Fujii Masao <masao.fujii@oss.nttdata.com> wrote: > Pushed. Thanks! > Thank you! -- John Naylor EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company