On Thu, Mar 1, 2018 at 3:03 AM, Andres Freund <andres@anarazel.de> wrote:
> - amcheck heap verification
>
> NR. Been submitted for a while, a modicum of code review has been
> performed.
>
> Adds a fair bit of code:
> 21 files changed, 1032 insertions(+), 64 deletions(-)
>
> but is low risk, being a readonly contrib module.
>
> Can probably committed if somebody has the energy.
A lot of that code is boilerplate test harness code. If you actually
look at the changes to amcheck, there are only a couple of hundred
lines of code, many of which are comments (there is also another 300
lines for the Bloom filter implementation). The changes to the
IndexBuildHeapScan() interface made by parallel CREATE INDEX allowed
me to simplify things considerably in the most recent revision.
--
Peter Geoghegan