ankitsultana commented on issue #16821: URL: https://github.com/apache/pinot/issues/16821#issuecomment-3300708414
I actually agree with that problem and was working on finding a solution for it for the Lite Mode when I encountered this issue. But the concern I have is that this is changing results for existing queries completely. e.g. if a user has `IS NOT NULL` in their MSE queries, before that filter was getting simplified to `true`. Now, the filter would actually be evaluated using null vectors at segment level (if they are enabled). Similarly, `IS NULL` would get simplified to `false` before but now will be executed for each segment, leading to different results. I'll need to check what the impact of this is at Uber, but if we do end up going with this approach we need to call this out as a breaking 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]
