cristianpop opened a new pull request, #18729:
URL: https://github.com/apache/pinot/pull/18729
Fix MSE null handling bug that leads to rows with null values being ignored
on queries such as the following:
```sql
SET useMultistageEngine=true;
SET enableNullHandling = true;
SELECT COUNT(*)
FROM test
WHERE prop IS NULL
OR prop = 'value'
```
The problem is that RexExpressionUtils::handleSearch doesn't take the null
as flag into consideration at all, so it gets lost.
The fix adds OR/AND for null checks besides the other filters when the null
as flag is set. Some unit tests were added to check the behaviour change.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
