From: | Amit Langote <amitlangote09(at)gmail(dot)com> |
---|---|
To: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: logical/relation.c header description |
Date: | 2020-09-18 03:36:54 |
Message-ID: | CA+HiwqG533En5SG+HYE+oU7cyO1wTy6sfLoa=v_5TQ2e-iod=g@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri, Sep 18, 2020 at 12:30 PM Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
> On Thu, Sep 17, 2020 at 6:46 PM Amit Langote <amitlangote09(at)gmail(dot)com> wrote:
> >
> > Thanks for taking a look.
> >
> > On Thu, Sep 17, 2020 at 9:20 PM Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
> > > On Thu, Sep 17, 2020 at 3:22 PM Amit Langote <amitlangote09(at)gmail(dot)com> wrote:
> > > >
> > >
> > > /*-------------------------------------------------------------------------
> > > * relation.c
> > > - * PostgreSQL logical replication
> > > + * PostgreSQL logical replication relation mapping routines
> > > *
> > > ..
> > > * NOTES
> > > * This file contains helper functions for logical replication relation
> > > * mapping cache.
> > >
> > > The new header title and NOTES section say the almost same thing. How
> > > about changing the title as "PostgreSQL logical replication relation
> > > mapping cache" and remove the NOTES section.
> >
> > That makes sense.
> >
> > Actually, I did consider expanding that NOTE to mention what the
> > module does but there are not actually that many interesting things in
> > there. If you would like to take a look at the text I had come up
> > with, please check the attached.
> >
> I am not very excited to add information about structures used in the
> file especially when we have explained 'LogicalRepPartMapEntry' a few
> lines after. I think we can leave explaining structures but otherwise,
> it looks good. See attached.
Works for me, thanks.
--
Amit Langote
EnterpriseDB: http://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2020-09-18 03:43:09 | Re: track_planning causing performance regression |
Previous Message | Amit Kapila | 2020-09-18 03:30:38 | Re: logical/relation.c header description |