I had a quick look at just the preliminary cleanup patches:
> 0001-BRIN-bloom-cleanup-20230218.patch
Looks good to me
> 0002-BRIN-minmax-multi-cleanup-20230218.patch
Looks good, although it would feel more natural to me to do it the other
way round, and define 'matches' as 'bool matches', and use DatumGetBool.
Not new with this patch, but I find the 'matches' and 'matching'
variables a bit strange. Wouldn't it be simpler to have just one variable?
> 0003-Introduce-bloom_filter_size-20230218.patch
Looks good
> 0004-Add-minmax-multi-inequality-tests-20230218.patch
Looks good
> +SELECT i/5 + mod(911 * i + 483, 25),
> + i/10 + mod(751 * i + 221, 41)
Peculiar formulas. Was there a particular reason for these values?
- Heikki