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)
--
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?