From: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
---|---|
To: | Amit Langote <amitlangote09(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:30:38 |
Message-ID: | CAA4eK1LX5ZQBSw1Ofreiq1fFEPxA6Xj413V_NXesO5NTkAbj+Q@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
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.
--
With Regards,
Amit Kapila.
Attachment | Content-Type | Size |
---|---|---|
logicalrep-file-header-desc_v3.patch | application/octet-stream | 1.3 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Langote | 2020-09-18 03:36:54 | Re: logical/relation.c header description |
Previous Message | Li Japin | 2020-09-18 03:00:14 | Re: Parallelize stream replication process |