smengcl commented on pull request #2946:
URL: https://github.com/apache/hadoop/pull/2946#issuecomment-825136131
> I think `hadoop.protobuf.version` in hadoop-project/pom.xml can be used
instead of defining the version in hadoop-yarn-csi module.
Good point! Done.
--
This is an auto
smengcl commented on pull request #2946:
URL: https://github.com/apache/hadoop/pull/2946#issuecomment-825206321
This is weird. The second commit essentially translate into the same thing
as the first one, but now it gets mvninstall -1 for Dependency convergence
error:
```
[WARNIN
smengcl commented on pull request #2946:
URL: https://github.com/apache/hadoop/pull/2946#issuecomment-826889878
Thanks @aajisaka . I have excluded protobuf-java 2.5.0 from yarn-csi
dependencies that introduced it.
--
This is an automated message from the Apache Git Service.
To respond to
smengcl commented on pull request #2946:
URL: https://github.com/apache/hadoop/pull/2946#issuecomment-828205946
@aajisaka Thanks for the reviewing and merging this.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the