Hi, I've pushed both the bloom and minmax-multi indexes today.
Based on the feedback and limitations described before I decided to keep them in core (i.e. not move them to contrib), but it was very useful experiment as it uncovered a couple issues - both in existing code, and in the definition of the new opclasses. After further thought I've concluded that the decision to ditch the old signature (in a1c649d889) was probably wrong, so I've added the support back, per what Alexander originally proposed. While doing so, I've noticed that the NULL handling may be a few bricks shy, because with (oi_regular_nulls == false) it should probably keep passing the NULL scan keys to the consistent function. I'll look into that and get it fixed. Thanks everyone who helped with those patches! -- Tomas Vondra EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company
