Re: [PGDOCS] Inconsistent linkends to "monitoring" views.

From: John Naylor <johncnaylorls(at)gmail(dot)com>
To: Peter Smith <smithpb2250(at)gmail(dot)com>
Cc: Michael Paquier <michael(at)paquier(dot)xyz>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: [PGDOCS] Inconsistent linkends to "monitoring" views.
Date: 2023-11-08 07:02:02
Message-ID: CANWCAZa5h38LDRkZRnGoGvXkiU3yWErxNpn1fQbnO4Ze4t4+SQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Oct 3, 2023 at 4:40 PM Peter Smith <smithpb2250(at)gmail(dot)com> wrote:
>
> On Tue, Oct 3, 2023 at 6:30 PM Michael Paquier <michael(at)paquier(dot)xyz> wrote:
> >
> > On Tue, Oct 03, 2023 at 01:11:15PM +1100, Peter Smith wrote:
> > > I noticed one or two "monitoring" links and linkends that are slightly
> > > inconsistent from all the others.
> >
> > - <link linkend="monitoring-pg-stat-subscription">
> > + <link linkend="monitoring-pg-stat-subscription-view">
> >
> > Is that really worth bothering for the internal link references?
>
> I preferred 100% consistency instead of 95% consistency. YMMV.
>
> > This can create extra backpatching conflicts.
>
> Couldn't the same be said for every patch that fixes a comment typo?
> This is like a link typo, so what's the difference?

My 2 cents: Comment typos are visible to readers, so more annoying
when seen in isolation, and less likely to have surroundings that
could change in back branches. Consistency would preferred all else
being equal, but then again nothing is wrong with the existing links.
In any case, no one has come out in favor of the patch, so it seems
like it should be rejected unless that changes.

--
John Naylor

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Drouvot, Bertrand 2023-11-08 07:02:34 Re: Synchronizing slots from primary to standby
Previous Message jian he 2023-11-08 06:56:09 Re: Infinite Interval