Thread

Commits

  1. Fix typo in rewriteheap.c.

  1. Fix a possible typo in rewriteheap.c code comments

    Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com> — 2022-01-10T04:12:37Z

    Hi,
    
    It looks like there's a typo, attaching a tiny patch to fix it.
    
      *
      * When doing logical decoding - which relies on using cmin/cmax of catalog
      * tuples, via xl_heap_new_cid records - heap rewrites have to log enough
    - * information to allow the decoding backend to updates its internal mapping
    + * information to allow the decoding backend to update its internal mapping
    
    Regards,
    Bharath Rupireddy.
    
  2. Re: Fix a possible typo in rewriteheap.c code comments

    Amit Kapila <amit.kapila16@gmail.com> — 2022-01-10T04:52:30Z

    On Mon, Jan 10, 2022 at 9:42 AM Bharath Rupireddy
    <bharath.rupireddyforpostgres@gmail.com> wrote:
    >
    > Hi,
    >
    > It looks like there's a typo, attaching a tiny patch to fix it.
    >
    >   *
    >   * When doing logical decoding - which relies on using cmin/cmax of catalog
    >   * tuples, via xl_heap_new_cid records - heap rewrites have to log enough
    > - * information to allow the decoding backend to updates its internal mapping
    > + * information to allow the decoding backend to update its internal mapping
    >
    
    LGTM.
    
    -- 
    With Regards,
    Amit Kapila.
    
    
    
    
  3. Re: Fix a possible typo in rewriteheap.c code comments

    Amit Kapila <amit.kapila16@gmail.com> — 2022-01-11T05:26:19Z

    On Mon, Jan 10, 2022 at 10:22 AM Amit Kapila <amit.kapila16@gmail.com> wrote:
    >
    > On Mon, Jan 10, 2022 at 9:42 AM Bharath Rupireddy
    > <bharath.rupireddyforpostgres@gmail.com> wrote:
    > >
    > > Hi,
    > >
    > > It looks like there's a typo, attaching a tiny patch to fix it.
    > >
    > >   *
    > >   * When doing logical decoding - which relies on using cmin/cmax of catalog
    > >   * tuples, via xl_heap_new_cid records - heap rewrites have to log enough
    > > - * information to allow the decoding backend to updates its internal mapping
    > > + * information to allow the decoding backend to update its internal mapping
    > >
    >
    > LGTM.
    >
    
    Pushed.
    
    -- 
    With Regards,
    Amit Kapila.