On 6/10/26 03:57, Enrique Sánchez wrote:
It's generally not safe to compare floating-point values for equality. I mean using `mcv_sel == 0.0` to detect that there are no extended MCV matches. IMO, we should use a bool variable (similar to how var_eq_const() does it) to track whether any MCV item matched, specifically in the loop where we check extended MCV matches in mcv_clauselist_selectivity(), and use that instead of checking `mcv_sel == 0.0`.
--
Best regards,
Ilia Evdokimov,
Tantor Labs LLC,
https://tantorlabs.com/
