RE: Use allocation macros in the logical replication code - Mailing list pgsql-hackers

Dear Peter,

Thanks for the patch. I have one comment for the patch:
```
@@ -836,8 +836,9 @@ SnapBuildAddCommittedTxn(SnapBuild *builder, TransactionId xid)
         elog(DEBUG1, "increasing space for committed transactions to %u",
              (uint32) builder->committed.xcnt_space);
 
-        builder->committed.xip = repalloc(builder->committed.xip,
-                                          builder->committed.xcnt_space * sizeof(TransactionId));
+        builder->committed.xip = repalloc_array(builder->committed.xip,
+                            TransactionId,
+                            builder->committed.xcnt_space);
```

Number of tabs seems not to enough here. Other than that LGTM.

Best regards,
Hayato Kuroda
FUJITSU LIMITED