shivjha30 opened a new pull request, #4879: URL: https://github.com/apache/hive/pull/4879
### What changes were proposed in this pull request? When executing queries via Beeline with the server-level execution engine set to MapReduce (MR) and the session-level engine set to Tez, the anticipated behavior involves the visibility of the Tez Progress bar. However, a discrepancy has been identified where the latest session value is not considered when instantiating the ProgressMonitorStatusMapper. This proposed pull request (PR) addresses the issue by retrieving the value of the session-level configuration for HIVE_EXECUTION_ENGINE, as opposed to the server-level configuration in service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIService.java. The modification ensures that the instantiation of the ProgressMonitorStatusMapper is appropriately evaluated, with the TezProgressMonitorStatusMapper being instantiated when the Hive execution engine is set to Tez at the session level. ### Why are the changes needed? When queries are executed through Beeline and the server-level execution engine is configured to MapReduce (MR), while the session-level execution engine is set to Tez, it has been observed that the Tez Progress bar is not rendered in the output. This goes against the expected behaviour for the Tez Progress Bar. It should be rendered in the output. ### Does this PR introduce _any_ user-facing change? No ### Is the change a dependency upgrade? No ### How was this patch tested? The test cases already exist. The visibility of the Progress Bar was tested by running Hive queries and changing the hive execution engine frequently to Tez and back to MR in several epochs. <img width="1840" alt="tez progress" src="https://github.com/apache/hive/assets/149884343/0672a932-7437-4f47-9584-08f529980064";> -- 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]
