itschrispeck opened a new pull request, #14299: URL: https://github.com/apache/pinot/pull/14299
I missed this edge case when adding TextMatchFilterOptimizer. For a query such as `text_match(col1, 'aaa') AND text_match(col1, 'bbb OR ccc')`, this would have previously been optimized to `text_match(col1, '(foo AND bar OR baz)'`, which would return unexpected results. Applying grouping for each filter (instead of at the merged filter level) produces an faithful translation to `text_match(col1, '(foo) AND (bar OR baz)'`. -- 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: commits-unsubscr...@pinot.apache.org 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
