ayushtkn commented on PR #493:
URL: https://github.com/apache/tez/pull/493#issuecomment-4364572573

   @abstractdog From the code point of view, this seems right, We had to just 
use the RPCEngine2 & migrate to the Hadoop Thirdparty. That is rightly done 
here.
   
   But I think this would be an incompatible change, beyond this point I 
believe Hive master won't build nor work with Tez master. Hive has to do the 
same activity when moving to Tez-1.0.0 & that would be incompatible for Hive as 
well & believe the older Hadoop versions won't work post this activity in Hive 
or Tez, unless there was something done later to fix it, which I don't think...
   
   There were compat issues that is why I believe HADOOP-17046 was done later 
to make sure older client can work with them. Means protobuf is indeed a very 
known heavy Incompat change, a lot of discussion at HADOOP-13363, in case 
interested. 
   
   These are the tradeoffs, I just wanted to highlight. Rest all cool


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

Reply via email to