On Fri, Jan 3, 2025 at 1:53 AM Matheus Alcantara <matheusssil...@gmail.com> wrote: > > > Yeap, I agree. Attached a v2 fixed. >
hi. some minor issue i found. +#include "storage/block.h" no need, since "#include "storage/bufmgr.h" already included it. do we need to add ``CHECK_FOR_INTERRUPTS()`` in heapam_read_stream_next_block? heapam_read_stream_next_block_private need add to src/tools/pgindent/typedefs.list overall, it looks good to me.