Re: dynamic monitoring

On Fri, Feb 21, 2025 at 1:18 PM Brad DeJong <bpd0018(at)gmail(dot)com> wrote:
>
> Hi all,
>
> In the section on dynamic monitoring, I found the following
> sentence confusing. I originally read it as "enclose the script"
> and "used to allow that too to be checked" instead of "enclose
> the script used" and "to allow that too to be checked".
>
> When discussing information found using dynamic tracing,
> be sure to enclose the script used to allow that too to be
> checked and discussed.
>
> Could it be rephrased as something like this?
>
> When discussing information found using dynamic tracing,
> be sure to enclose the script used to find the information
> so that both the script and the information can be checked
> and discussed.
>
> Regards,
> Brad
>
> C:\src\postgresql\doc\src\sgml>git diff -p
> diff --git a/doc/src/sgml/monitoring.sgml b/doc/src/sgml/monitoring.sgml
> index 71c4f96d05..d89261326a 100644
> --- a/doc/src/sgml/monitoring.sgml
> +++ b/doc/src/sgml/monitoring.sgml
> @@ -7428,7 +7428,8 @@ Total time (ns) 2312105013
> be meaningless. In most cases where problems are found it is the
> instrumentation that is at fault, not the underlying system. When
> discussing information found using dynamic tracing, be sure to enclose
> - the script used to allow that too to be checked and discussed.
> + the script used to find that information so that both the script
> + and the information can be checked and discussed.
> </para>
> </sect2>

+1 from me.

Robert Treat
https://xzilla.net