On Wed, Jun 16, 2021 at 3:51 PM Amit Kapila <amit.kapil...@gmail.com> wrote: > > On Mon, Jun 14, 2021 at 5:33 PM osumi.takami...@fujitsu.com > <osumi.takami...@fujitsu.com> wrote: > > > > On Friday, June 11, 2021 2:13 PM vignesh C <vignes...@gmail.com> wrote: > > > > Attached the patch-set that addressed those two comments. > > > > Few minor comments: > 1. > + <listitem> > + <para> > + Clustering <structname>pg_class</structname> in a transaction. > > Can we change above to: Perform <command>CLUSTER</command> on > <structname>pg_class</structname> in a transaction. > > 2. > + <listitem> > + <para> > + Executing <command>TRUNCATE</command> on user catalog table > in a transaction. > + </para> > > Square brackets are missing for user. > > 3. > + <indexterm> > + <primary>Overview</primary> > + </indexterm> > .. > .. > + <indexterm> > + <primary>Caveats</primary> > + </indexterm> > > Why are these required when we already have titles? I have seen other > places in the docs where we use titles for Overview and Caveats but > they didn't have similar usage. >
Even I felt this was not required. I had checked other places and also prepared doc by removing it, it works fine. Regards, Vignesh
