Re: descriptions of pg_stat_user_functions and pg_stat_slru

On 2020/05/25 14:23, Fujii Masao wrote:
> 
> 
> On 2020/05/22 22:35, Fujii Masao wrote:
>>
>>
>> On 2020/05/21 4:53, Tom Lane wrote:
>>> Fujii Masao  writes:
>>>> On 2020/05/20 22:32, Tom Lane wrote:
>>>>> OK by me --- that, too, would be more like the existing catalogs
>>>>> chapter.
>>>
>>>> Yeah, so I'd like to propose the attached patch.
>>>
>>> Hmmm ... I'm not exactly convinced about sticking xreflabels onto
>>> the s as you've done here.  Presumably that would make s
>>> render like "pg_stat_slru" not "Section 27.2.3", which I think is
>>> not consistent with our practice elsewhere.  I'd be inclined to
>>> leave the id attributes on the s, and add xreflabels there
>>> if we want them.
>>>
>>> I see that catalogs.sgml doesn't really match either of those approaches,
>>> though.  Not sure if we want to change it.  It looks like people have
>>> tended to use  to substitute text for xref's to the catalog
>>> sections, so maybe it would be better to add xreflabels there too
>>> and simplify the references.
>>
>> Yeah, since I think that using  is simpler than , I added xlabel
>> in . But if we don't do that for the consistency with catalog.sgml,
>> I think that there are two approaches.
>>
>> (1) Replace  with  when referencing to the monitoring views
>>         docs. For example, add 
>>         and replace  with
>>         .
>>
>> (2) Leave  as it is. In this case, for example,
>>          references to the table
>>         of pg_stat_replication instead of the section.
>>
>> I prefer (1) because it's better to reference to the section rather than
>> the table. There are thirty  for monitoring views in the docs and
>> they need to be updated.
> 
> Attached (monitoring_docfix_v2.patch) is the patch for (1).

Barring any objection, I will commit this patch.

Regards,

-- 
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION