> On Nov 24, 2025, at 16:07, Chao Li <[email protected]> wrote:
>
> 0006, 0007 look good, no comment.
I missed a nit comment in 0007:
7 - 0007
```
+ * To handle recovery conflict during logical decoding on standby, we must know
+ * if the table is a catalog table. Note that in visibilitymapdefs.h
+ * VISIBLITYMAP_XLOG_CATALOG_REL is also defined as (1 << 2). xl_heap_prune
+ * records should use XLHP_IS_CATALOG_REL, not VISIBILIYTMAP_XLOG_CATALOG_REL
--
+ * even if they only contain updates to the VM.
```
VISIBLITYMAP_XLOG_CATALOG_REL missed “I” after “B”.
Best regards,
--
Chao Li (Evan)
HighGo Software Co., Ltd.
https://www.highgo.com/
