Hi Andrey,

I think that you forgot to attach the patch with the actual fix?

-- 
Peter Geoghegan

On Sun, Aug 2, 2026 at 2:12 AM Andrey Borodin <[email protected]> wrote:
>
>
>
> > On 1 Aug 2026, at 20:01, Peter Geoghegan <[email protected]> wrote:
> >
> > <0001-Add-regression-test-for-multirange-GiST-containment-.patch>
>
> Hi Peter,
>
> I agree with the test and also think this is a bug.  The check in
> multirange_gist_consistent() is too strict and rejects tuples that
> should reach the recheck.  I think we should use
> range_gist_consistent_int_multirange() for this case.  Using the
> internal predicate for every strategy would also be correct, but would
> weaken filtering and, IMO, cause unnecessary heap rechecks.
>
> PFA v2 patch set.  The first patch is your test, and the second is the
> fix as I see it.
>
> Thanks!
>
>
> Best regards, Andrey Borodin.



-- 
Peter Geoghegan


Reply via email to