deniskuzZ commented on PR #5033: URL: https://github.com/apache/hive/pull/5033#issuecomment-1908338064
> why do we have this copy of ProtoMessageWritable in tez package namespace in hive code? @harishjp, do you know anything about this? > > https://github.com/apache/tez/blob/master/tez-plugins/tez-protobuf-history-plugin/src/main/java/org/apache/tez/dag/history/logging/proto/ProtoMessageWritable.java see [HIVE-19288](https://issues.apache.org/jira/browse/HIVE-19288), looks like at that time copied code was in unreleased version of Hive. Now since it's available we can just drop the copy and use Tez libs. That said, I think, patch should be done in Tez -- 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]
