achennagiri opened a new pull request, #3487: URL: https://github.com/apache/hive/pull/3487
### What changes were proposed in this pull request? This ticket is an improvement on https://issues.apache.org/jira/browse/HIVE-26426. The canHandleQbForCbo() checks whether Calcite can handle the query or not and it returns null if the query can be handled; non-null reason string if it cannot be. But currently, it returns an empty string if INFO Log is not enabled which should not be the case. Also, this is probably a performance optimization that is not needed and can be simplified. ### Why are the changes needed? Simplify the code and improve the correctness. ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Unit tests. -- 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]
