fix: correct OR null semantics for nullable masks

[fenfeng9]

closes #5895

Allow|Block OR previously assumed NULL rows were always included in the block.selected set; when they were not, NULLs could be dropped and FALSE/NULL were mixed, leading to incorrect results.

This change computes TRUE/FALSE/NULL explicitly for Allow|Block OR and derives NULL/selected sets from those.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[fenfeng9]

Hi @wjones127, could you please review this when you have a moment? Thanks!

[wjones127]

Could you add this test to https://github.com/lance-format/lance/blob/main/rust/lance/tests/query/primitives.rs ? That's where we want these kind of query tests.

I didn't ask for this on the label list PRs because we don't yet have a nested field test suite merged. I've started them here: #5901

Should just be able to add a filter or two.

[fenfeng9]

Could you add this test to https://github.com/lance-format/lance/blob/main/rust/lance/tests/query/primitives.rs ? That's where we want these kind of query tests.

I didn't ask for this on the label list PRs because we don't yet have a nested field test suite merged. I've started them here: #5901

Should just be able to add a filter or two.

Sure, I'll handle this tomorrow. Should I keep the test in Python?

[wjones127]

Could you add this test to https://github.com/lance-format/lance/blob/main/rust/lance/tests/query/primitives.rs ? That's where we want these kind of query tests.
I didn't ask for this on the label list PRs because we don't yet have a nested field test suite merged. I've started them here: #5901
Should just be able to add a filter or two.

Sure, I'll handle this tomorrow. Should I keep the test in Python?

You can if you want.

[wjones127]

Looks good. Thank you!