abstractdog commented on PR #4879:
URL: https://github.com/apache/hive/pull/4879#issuecomment-1835754039

   > MR is not deprecated nor discontinued as an execution engine in Apache 
Hive, it is a Cloudera thing for us :-)
   > 
   > This patch makes sense to me, anyway if the job was executed via Tez, we 
should display the Tez Progress bar.
   > 
   > Regarding the changing of execution engine at Session level, that is also 
fine, right? that is users choice, anyway I believe that behaviour is 
independent of this PR.
   > 
   > @shivjha30 you need to get a green build, there is some unrelated test 
failure
   
   yeah, maybe officially we haven't deprecated MR, which is a big mistake, but 
that's another topic
   I'm fine with forcing the session conf, as that's what reflects the actual 
user's intention


-- 
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]

Reply via email to