Thread: dynamic monitoring

On Fri, Feb 21, 2025 at 1:18 PM Brad DeJong <bpd0018@gmail.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