On Tue, Mar 24, 2020 at 10:06 PM Ibrar Ahmed <ibrar.ah...@gmail.com> wrote:
>
>
> On Fri, Mar 13, 2020 at 6:58 AM Justin Pryzby <pry...@telsasoft.com>
> wrote:
>
>>
>> Thanks for picking up this patch. There's a minor typo:
>>
>> + * readable outside of this sessoin. Therefore
>> doing IO here isn't
>>
>> => session
>>
>> --
>> Justin
>>
>
> Thanks, please see the updated and rebased patch. (master
> 17a28b03645e27d73bf69a95d7569b61e58f06eb)
>
> --
> Ibrar Ahmed
>
Andres while fixing the one FIXME in the patch
" visibilitymap_pin(relation, BufferGetBlockNumber(buffer),
&vmbuffer);
/*
* FIXME: setting recptr here is a dirty dirty hack, to prevent
* visibilitymap_set() from WAL logging.
*
"
I am not able to see any scenario where recptr is not set before reaching
to that statement. Can you clarify why you think recptr will not be set at
that statement?
--
Ibrar Ahmed
