Hi,
> Note that I have not tried any of this yet.
I did, both with Meson and Autotools. All in all the patch looks very
good, but I have a few little nitpicks.
```
+ /* HOT chains should not intersect. */
+ if (predecessor[nextoffnum] != InvalidOffsetNumber)
+ {
+ report_corruption(&ctx,
+ psprintf("redirect line pointer
points to offset %u, but offset %u also points there",
+ (unsigned) nextoffnum,
(unsigned) predecessor[nextoffnum]));
+ continue;
+ }
```
This type of corruption doesn't seem to be test-covered.
```
+ /*
+ * If the next line pointer is a redirect, or if it's a tuple
+ * but the XMAX of this tuple doesn't match the XMIN of the next
+ * tuple, then the two aren't part of the same update chain and
+ * there is nothing more to do.
+ */
+ if (ItemIdIsRedirected(next_lp))
+ continue;
```
lcov shows that the `continue` path is never executed. This is
probably not a big deal however.
```
+$node->append_conf('postgresql.conf','max_prepared_transactions=100');
```
From what I can tell this line is not needed.
--
Best regards,
Aleksander Alekseev