ShubhamChaurasia opened a new pull request #2318: URL: https://github.com/apache/hive/pull/2318
The issue with order by in llap external client is described in https://issues.apache.org/jira/browse/HIVE-25159 ### What changes were proposed in this pull request? To resolve the above issue, we propose removing of order by handling in the llap external client library and let the clients do it at their end. ### Why are the changes needed? 1. These changes are needed to make llap external client queries more performant, earlier they used to generate only a single split for order by queries. 2. Removing order by support also helps us to get rid of exception arising to due false detection of order by clause (it was specific to spark-llap and was done for that only). ### Does this PR introduce _any_ user-facing change? It removes a config - hive.llap.external.splits.order.by.force.single.split ### How was this patch tested? Through existing unit tests - org.apache.hive.jdbc.TestJdbcGenericUDTFGetSplits - org.apache.hive.jdbc.TestJdbcGenericUDTFGetSplits2 -- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
