|From:||Michael Paquier <michael(dot)paquier(at)gmail(dot)com>|
|To:||Jeff Janes <jeff(dot)janes(at)gmail(dot)com>|
|Subject:||Re: Monitoring of a hot standby with a largely idle master|
|Views:||Raw Message | Whole Thread | Download mbox|
On Mon, Jul 24, 2017 at 9:08 PM, Jeff Janes <jeff(dot)janes(at)gmail(dot)com> wrote:
> On Sun, Jul 16, 2017 at 8:47 AM, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
>> What do you think about the patch attached?
> Looks OK. Should it mention specifically "On a hot standby" rather than "On
> a standby"? Otherwise people might be left confused on how they are
> supposed to do this on a generic standby. It is the kind of thing which is
> obvious once you know it, but confusing the first time you encounter it.
Yes, right. Let's update as you suggest.
>> You can retrieve a list of WAL sender processes via the
>> - <link linkend="monitoring-stats-views-table">
>> + <link linkend="monitoring-stats-dynamic-views-table">
>> <literal>pg_stat_replication</></link> view. Large differences
>> In the previous paragraph I have noticed that the link reference is
>> incorrect. pg_stat_replication is listed under
> Yes, that is clearly wrong. But why not link directly to the description of
> the view itself, pg-stat-replication-view, rather than the correct table
> which mentions the view? Is that the accepted docs style to link to the
> more generic place? (Same thing applies to your patch, it could link
> directly to pg-stat-wal-receiver-view.
Yes, that's even better.
> Sorry for the delay, it took me awhile to get the new doc build system to
> work (solution seems to be, "Don't use CentOS6 anymore")
Thanks for the review.
|Next Message||PAWAN SHARMA||2017-07-26 08:59:44||Fwd: getting error while parsing log file using pgbadger|
|Previous Message||Nick Brennan||2017-07-26 05:34:15||Indexes being ignored after upgrade to 9.5|