pg.ddx.io pgsql-docs@postgresql.org mailing list archive
help / color / mirror / Atom feedRE: index items for pg_stat_progress_xxx views
6+ messages / 2 participants
[nested] [flat]
* RE: index items for pg_stat_progress_xxx views
@ 2020-11-30 01:57 Shinya11.Kato@nttdata.com
2020-11-30 06:11 ` Re: index items for pg_stat_progress_xxx views Fujii Masao <masao.fujii@oss.nttdata.com>
0 siblings, 1 reply; 6+ messages in thread
From: Shinya11.Kato@nttdata.com @ 2020-11-30 01:57 UTC (permalink / raw)
To: masao.fujii@oss.nttdata.com; pgsql-docs@lists.postgresql.org
> Hi,
>
> The index items for pg_stat_progress_xxx views point to the "Viewing
> Statistics" section, but not to the dedicated section (e.g., "ANALYZE
> Progress
> Reporting") for each view. IMO this is very inconvenient when finding
> the section describing each pg_stat_progress_xxx view, from the index.
> So what about adding new pointer to the section for each view in the index?
> Patch attached.
>
> BTW, other stats views have both pointers in the index.
When setting an <indexterm>, it's better to set the zone attribute to indicate which section it's associated with.
For example,
<indexterm zone="analyze-progress-reporting">
<primary>pg_stat_progress_analyze</primary>
</indexterm>
Regards,
Shinya Kato
^ permalink raw reply [nested|flat] 6+ messages in thread
* Re: index items for pg_stat_progress_xxx views
2020-11-30 01:57 RE: index items for pg_stat_progress_xxx views Shinya11.Kato@nttdata.com
@ 2020-11-30 06:11 ` Fujii Masao <masao.fujii@oss.nttdata.com>
2020-12-01 04:26 ` RE: index items for pg_stat_progress_xxx views Shinya11.Kato@nttdata.com
0 siblings, 1 reply; 6+ messages in thread
From: Fujii Masao @ 2020-11-30 06:11 UTC (permalink / raw)
To: Shinya11.Kato@nttdata.com; pgsql-docs@lists.postgresql.org
On 2020/11/30 10:57, Shinya11.Kato@nttdata.com wrote:
>> Hi,
>>
>> The index items for pg_stat_progress_xxx views point to the "Viewing
>> Statistics" section, but not to the dedicated section (e.g., "ANALYZE
>> Progress
>> Reporting") for each view. IMO this is very inconvenient when finding
>> the section describing each pg_stat_progress_xxx view, from the index.
>> So what about adding new pointer to the section for each view in the index?
>> Patch attached.
>>
>> BTW, other stats views have both pointers in the index.
>
> When setting an <indexterm>, it's better to set the zone attribute to indicate which section it's associated with.
>
> For example,
> <indexterm zone="analyze-progress-reporting">
> <primary>pg_stat_progress_analyze</primary>
> </indexterm>
Thanks for the review!
I was thinking that the zone attribute doesn't need to be specified
because the indexterm is defined under the section that the indexterm
tries to point to. You can see the same situation at, for example,
the section and indexterm of pg_stat_database. If I'm missing something,
could you tell me why the zone attribute is necessary in this case?
Regards,
--
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION
^ permalink raw reply [nested|flat] 6+ messages in thread
* RE: index items for pg_stat_progress_xxx views
2020-11-30 01:57 RE: index items for pg_stat_progress_xxx views Shinya11.Kato@nttdata.com
2020-11-30 06:11 ` Re: index items for pg_stat_progress_xxx views Fujii Masao <masao.fujii@oss.nttdata.com>
@ 2020-12-01 04:26 ` Shinya11.Kato@nttdata.com
2020-12-01 05:06 ` Re: index items for pg_stat_progress_xxx views Fujii Masao <masao.fujii@oss.nttdata.com>
0 siblings, 1 reply; 6+ messages in thread
From: Shinya11.Kato@nttdata.com @ 2020-12-01 04:26 UTC (permalink / raw)
To: masao.fujii@oss.nttdata.com; pgsql-docs@lists.postgresql.org
>>> Hi,
>>>
>>> The index items for pg_stat_progress_xxx views point to the "Viewing
>>> Statistics" section, but not to the dedicated section (e.g., "ANALYZE
>>> Progress
>>> Reporting") for each view. IMO this is very inconvenient when finding
>>> the section describing each pg_stat_progress_xxx view, from the index.
>>> So what about adding new pointer to the section for each view in the index?
>>> Patch attached.
>>>
>>> BTW, other stats views have both pointers in the index.
>>
>> When setting an <indexterm>, it's better to set the zone attribute to indicate
>which section it's associated with.
>>
>> For example,
>> <indexterm zone="analyze-progress-reporting">
>> <primary>pg_stat_progress_analyze</primary>
>> </indexterm>
>
>Thanks for the review!
>
>I was thinking that the zone attribute doesn't need to be specified because the
>indexterm is defined under the section that the indexterm tries to point to. You
>can see the same situation at, for example, the section and indexterm of
>pg_stat_database. If I'm missing something, could you tell me why the zone
>attribute is necessary in this case?
I looked at lines 132-137 of monitoring.sgml, and I saw that zone is set for indexterm after <sect1>.
However, I haven't found the zone set for indexterm after <sect2>, so you may be right.
Regards,
Shinya Kato
^ permalink raw reply [nested|flat] 6+ messages in thread
* Re: index items for pg_stat_progress_xxx views
2020-11-30 01:57 RE: index items for pg_stat_progress_xxx views Shinya11.Kato@nttdata.com
2020-11-30 06:11 ` Re: index items for pg_stat_progress_xxx views Fujii Masao <masao.fujii@oss.nttdata.com>
2020-12-01 04:26 ` RE: index items for pg_stat_progress_xxx views Shinya11.Kato@nttdata.com
@ 2020-12-01 05:06 ` Fujii Masao <masao.fujii@oss.nttdata.com>
2020-12-01 05:14 ` RE: index items for pg_stat_progress_xxx views Shinya11.Kato@nttdata.com
0 siblings, 1 reply; 6+ messages in thread
From: Fujii Masao @ 2020-12-01 05:06 UTC (permalink / raw)
To: Shinya11.Kato@nttdata.com; pgsql-docs@lists.postgresql.org
On 2020/12/01 13:26, Shinya11.Kato@nttdata.com wrote:
>>>> Hi,
>>>>
>>>> The index items for pg_stat_progress_xxx views point to the "Viewing
>>>> Statistics" section, but not to the dedicated section (e.g., "ANALYZE
>>>> Progress
>>>> Reporting") for each view. IMO this is very inconvenient when finding
>>>> the section describing each pg_stat_progress_xxx view, from the index.
>>>> So what about adding new pointer to the section for each view in the index?
>>>> Patch attached.
>>>>
>>>> BTW, other stats views have both pointers in the index.
>>>
>>> When setting an <indexterm>, it's better to set the zone attribute to indicate
>> which section it's associated with.
>>>
>>> For example,
>>> <indexterm zone="analyze-progress-reporting">
>>> <primary>pg_stat_progress_analyze</primary>
>>> </indexterm>
>>
>> Thanks for the review!
>>
>> I was thinking that the zone attribute doesn't need to be specified because the
>> indexterm is defined under the section that the indexterm tries to point to. You
>> can see the same situation at, for example, the section and indexterm of
>> pg_stat_database. If I'm missing something, could you tell me why the zone
>> attribute is necessary in this case?
>
> I looked at lines 132-137 of monitoring.sgml, and I saw that zone is set for indexterm after <sect1>.
> However, I haven't found the zone set for indexterm after <sect2>, so you may be right.
So you agree not to add zone attribute in this case?
Regards,
--
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION
^ permalink raw reply [nested|flat] 6+ messages in thread
* RE: index items for pg_stat_progress_xxx views
2020-11-30 01:57 RE: index items for pg_stat_progress_xxx views Shinya11.Kato@nttdata.com
2020-11-30 06:11 ` Re: index items for pg_stat_progress_xxx views Fujii Masao <masao.fujii@oss.nttdata.com>
2020-12-01 04:26 ` RE: index items for pg_stat_progress_xxx views Shinya11.Kato@nttdata.com
2020-12-01 05:06 ` Re: index items for pg_stat_progress_xxx views Fujii Masao <masao.fujii@oss.nttdata.com>
@ 2020-12-01 05:14 ` Shinya11.Kato@nttdata.com
2020-12-01 08:21 ` Re: index items for pg_stat_progress_xxx views Fujii Masao <masao.fujii@oss.nttdata.com>
0 siblings, 1 reply; 6+ messages in thread
From: Shinya11.Kato@nttdata.com @ 2020-12-01 05:14 UTC (permalink / raw)
To: masao.fujii@oss.nttdata.com; pgsql-docs@lists.postgresql.org
>>>>> Hi,
>>>>>
>>>>> The index items for pg_stat_progress_xxx views point to the
>>>>> "Viewing Statistics" section, but not to the dedicated section
>>>>> (e.g., "ANALYZE Progress
>>>>> Reporting") for each view. IMO this is very inconvenient when
>>>>> finding the section describing each pg_stat_progress_xxx view, from the
>index.
>>>>> So what about adding new pointer to the section for each view in the
>index?
>>>>> Patch attached.
>>>>>
>>>>> BTW, other stats views have both pointers in the index.
>>>>
>>>> When setting an <indexterm>, it's better to set the zone attribute
>>>> to indicate
>>> which section it's associated with.
>>>>
>>>> For example,
>>>> <indexterm zone="analyze-progress-reporting">
>>>> <primary>pg_stat_progress_analyze</primary>
>>>> </indexterm>
>>>
>>> Thanks for the review!
>>>
>>> I was thinking that the zone attribute doesn't need to be specified
>>> because the indexterm is defined under the section that the indexterm
>>> tries to point to. You can see the same situation at, for example,
>>> the section and indexterm of pg_stat_database. If I'm missing
>>> something, could you tell me why the zone attribute is necessary in this case?
>>
>> I looked at lines 132-137 of monitoring.sgml, and I saw that zone is set for
>indexterm after <sect1>.
>> However, I haven't found the zone set for indexterm after <sect2>, so you may
>be right.
>
>So you agree not to add zone attribute in this case?
Yes, I think it's nice.
Regards,
Shinya Kato
^ permalink raw reply [nested|flat] 6+ messages in thread
* Re: index items for pg_stat_progress_xxx views
2020-11-30 01:57 RE: index items for pg_stat_progress_xxx views Shinya11.Kato@nttdata.com
2020-11-30 06:11 ` Re: index items for pg_stat_progress_xxx views Fujii Masao <masao.fujii@oss.nttdata.com>
2020-12-01 04:26 ` RE: index items for pg_stat_progress_xxx views Shinya11.Kato@nttdata.com
2020-12-01 05:06 ` Re: index items for pg_stat_progress_xxx views Fujii Masao <masao.fujii@oss.nttdata.com>
2020-12-01 05:14 ` RE: index items for pg_stat_progress_xxx views Shinya11.Kato@nttdata.com
@ 2020-12-01 08:21 ` Fujii Masao <masao.fujii@oss.nttdata.com>
0 siblings, 0 replies; 6+ messages in thread
From: Fujii Masao @ 2020-12-01 08:21 UTC (permalink / raw)
To: Shinya11.Kato@nttdata.com; pgsql-docs@lists.postgresql.org
On 2020/12/01 14:14, Shinya11.Kato@nttdata.com wrote:
>>>>>> Hi,
>>>>>>
>>>>>> The index items for pg_stat_progress_xxx views point to the
>>>>>> "Viewing Statistics" section, but not to the dedicated section
>>>>>> (e.g., "ANALYZE Progress
>>>>>> Reporting") for each view. IMO this is very inconvenient when
>>>>>> finding the section describing each pg_stat_progress_xxx view, from the
>> index.
>>>>>> So what about adding new pointer to the section for each view in the
>> index?
>>>>>> Patch attached.
>>>>>>
>>>>>> BTW, other stats views have both pointers in the index.
>>>>>
>>>>> When setting an <indexterm>, it's better to set the zone attribute
>>>>> to indicate
>>>> which section it's associated with.
>>>>>
>>>>> For example,
>>>>> <indexterm zone="analyze-progress-reporting">
>>>>> <primary>pg_stat_progress_analyze</primary>
>>>>> </indexterm>
>>>>
>>>> Thanks for the review!
>>>>
>>>> I was thinking that the zone attribute doesn't need to be specified
>>>> because the indexterm is defined under the section that the indexterm
>>>> tries to point to. You can see the same situation at, for example,
>>>> the section and indexterm of pg_stat_database. If I'm missing
>>>> something, could you tell me why the zone attribute is necessary in this case?
>>>
>>> I looked at lines 132-137 of monitoring.sgml, and I saw that zone is set for
>> indexterm after <sect1>.
>>> However, I haven't found the zone set for indexterm after <sect2>, so you may
>> be right.
>>
>> So you agree not to add zone attribute in this case?
>
> Yes, I think it's nice.
So I pushed the patch. Thanks!
Regards,
--
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION
^ permalink raw reply [nested|flat] 6+ messages in thread
end of thread, other threads:[~2020-12-01 08:21 UTC | newest]
Thread overview: 6+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2020-11-30 01:57 RE: index items for pg_stat_progress_xxx views Shinya11.Kato@nttdata.com
2020-11-30 06:11 ` Fujii Masao <masao.fujii@oss.nttdata.com>
2020-12-01 04:26 ` Shinya11.Kato@nttdata.com
2020-12-01 05:06 ` Fujii Masao <masao.fujii@oss.nttdata.com>
2020-12-01 05:14 ` Shinya11.Kato@nttdata.com
2020-12-01 08:21 ` Fujii Masao <masao.fujii@oss.nttdata.com>
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox