mbathori-cloudera commented on PR #192: URL: https://github.com/apache/tez/pull/192#issuecomment-1145335453
@abstractdog I replaced the protobuf install and compile plugin with [protoc-jar-maven-plugin](http://os72.github.io/protoc-jar-maven-plugin/) as [slachiewicz](https://github.com/slachiewicz) recommended it. It handles the plugin download and compilation so I think we can get rid of the `install-protobuf.sh` script. It also supports version 2.5.0 so we can keep the backward compatibility. I’ve run a couple of test build locally for different versions and they worked properly. Unfortunately the PR build won't succeed on this either, since it still use the master branch for compilation without the PR changes. What do you think about this change? -- 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]
