Re: Add docs stub for recovery.conf
Stephen Frost <sfrost@snowman.net>
From: Stephen Frost <sfrost@snowman.net>
To: Euler Taveira <euler@eulerto.com>
Cc: Craig Ringer <craig.ringer@enterprisedb.com>, Bruce Momjian <bruce@momjian.us>, Isaac Morland <isaac.morland@gmail.com>, "David G. Johnston" <david.g.johnston@gmail.com>, pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2021-03-28T21:20:41Z
Lists: pgsql-hackers
Attachments
- doc-rename-cleanup_v5.patch (text/x-diff) patch v5
- doc-rename-cleanup_v5_v10.patch (text/x-diff) patch v5
- doc-rename-cleanup_v5_v11.patch (text/x-diff) patch v5
- doc-rename-cleanup_v5_v12.patch (text/x-diff) patch v5
- doc-rename-cleanup_v5_v13.patch (text/x-diff) patch v5
Greetings, * Euler Taveira (euler@eulerto.com) wrote: > On Sun, Mar 28, 2021, at 2:22 PM, Stephen Frost wrote: > > Unless there's anything further, will commit these soon. > I briefly looked at this patch and have a few comments. > > +<sect1 id="app-pgreceivexlog" xreflabel="pg_receivexlog"> > + <title><command>pg_receivexlog</command> renamed to <command>pg_recievewal</command></title> > > s/pg_recievewal/pg_receivewal/ Hah! Good catch. Fixed. > +<appendix id="appendix-obsolete"> > + <title>Obsolete or renamed features, settings and files</title> > > Section titles are capitalized so it should be "Obsolete or Renamed Features, > Settings and Files". I find this section name too descriptive. I didn't follow > this thread but maybe we might use a generic name that is also shorter than it > such as "Incompatible or Obsolete Features". Not sure how others feel, but 'incompatible' doesn't seem quite right to me- we renamed some things between major versions but they're as compatible as they were before. Having it be shorter does make sense to me, so I updated the patch with "Obsolete or Renamed Features" instead. I also reworded the section underneath a bit to mention renaming and to include a comment about features, settings, and file names. > + <primary>trigger_file</primary> > + <see>promote_trigger_file</see> > + </indexterm> > + setting has been renamed to > + <xref linkend="guc-promote-trigger-file"/> > + </para> > > There should be a period after the <xref>. Good catch, fixed. Updated patches attached. Thanks! Stephen
Commits
-
Add a docs section for obsoleted and renamed functions and settings
- 3b0c647bbfc5 14.0 landed
- 89e383b30a94 13.3 landed
- 448e86c3345b 12.7 landed
- 625f5aae7e42 11.12 landed
- 58960fca4ef1 10.17 landed