Thread: document pg_settings view doesn't display custom options
Starting separate threads to keep from cluttering the TODO list thread.
Here's a patch for the subject, as mentioned in
Attachment
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
Attachment
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
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.
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.
--
<link linkend="runtime-config-custom">customized options</link>
that have been set before the relevant extension module has been loaded.
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
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?
"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..."
The pg_settings view..."
--
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
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.
Attachment
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
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?
Attachment
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
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
On Mon, Nov 9, 2020 at 2:12 AM Fujii Masao <masao.fujii@oss.nttdata.com> wrote:
Pushed. Thanks!
Thank you!