From: | Matheus Alcantara <matheusssilv97(at)gmail(dot)com> |
---|---|
To: | vignesh C <vignesh21(at)gmail(dot)com> |
Cc: | jian he <jian(dot)universality(at)gmail(dot)com>, Kirill Reshke <reshkekirill(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: read stream on amcheck |
Date: | 2025-03-17 13:46:32 |
Message-ID: | CAFY6G8c34G5pwv32iEJa77m63MC63Kxmvn==kz8+FVgFLxGOEg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi,
On Sun, Mar 16, 2025 at 10:30 AM vignesh C <vignesh21(at)gmail(dot)com> wrote:
>
> On Wed, 12 Feb 2025 at 00:11, Matheus Alcantara
> <matheusssilv97(at)gmail(dot)com> wrote:
> >
> > Hi,
> >
> > Em ter., 11 de fev. de 2025 às 03:39, jian he
> > <jian(dot)universality(at)gmail(dot)com> escreveu:
> > > hi. some minor issue i found.
> > >
> > > +#include "storage/block.h"
> > > no need, since "#include "storage/bufmgr.h" already included it.
> > >
> > Fixed
> >
> > > do we need to add ``CHECK_FOR_INTERRUPTS()`` in heapam_read_stream_next_block?
> > >
> > The current code on master don't CHECK_FOR_INTERRUPTS, so I would prefer to not
> > change this behaviour, but I think that is considerable.
> >
> > > heapam_read_stream_next_block_private need add to
> > > src/tools/pgindent/typedefs.list
> > >
> > Fixed
> >
> > > overall, it looks good to me.
> >
> > Thanks for the review! v3 with the fixes attached.
>
> I noticed that Nazir Bilal Yavuz's comment from [1] is not yet
> addressed, i have changed the status of commitfest entry to Waiting on
> Author, please address them and update it to Needs review.
> [1] - https://www.postgresql.org/message-id/CAN55FZ2CGxcqTk_LLRPAi2aFNqtR4o=JPfjN0=yT0yObfQ2h2g@mail.gmail.com
>
Sorry for the delay, attached v4 with the remaining fixes.
--
Matheus Alcantara
Attachment | Content-Type | Size |
---|---|---|
v4-0001-Use-read-stream-on-amcheck.patch | application/octet-stream | 4.9 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2025-03-17 13:52:41 | Re: making EXPLAIN extensible |
Previous Message | Dmitry Koval | 2025-03-17 13:36:56 | Re: Add SPLIT PARTITION/MERGE PARTITIONS commands |