siddharthteotia opened a new pull request #5643: URL: https://github.com/apache/incubator-pinot/pull/5643
Two fixes: - Make in-built functions like TEXT_MATCH, REGEXP_LIKE case insensitive on SQL path. Should be converted to upper case since that is what is there in enum definitions and is also followed by Calcite for all functions/operators part of it's grammar. - Add support for converting PinotQuery to BrokerRequest for IS NULL and IS NOT NULL. Calcite supports these natively as part of it's grammar but we missed handling them when converting FilterExpression to FilterQuery. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
