ankitsultana commented on issue #16821: URL: https://github.com/apache/pinot/issues/16821#issuecomment-3300617136
> When null handling is disabled must be transformed into: I don't follow, why should the `NOT IS NULL` get transformed to `<> 0`? Prior to this change if column level nullability was disabled, the user contract for MSE was that each column would be considered as not nullable, so it is correct to simplify `NOT IS NULL` to `true`. Moreover, the default value in the schema shouldn't be used in such a way in the query optimizer. IMO that default value has only served as a guide for the storage layer, and it can change across segments. -- 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]
