[GitHub] [hadoop] smengcl commented on pull request #2946: YARN-10747. Bump YARN CSI protobuf version to 3.7.1

2021-04-22 Thread GitBox
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

[GitHub] [hadoop] smengcl commented on pull request #2946: YARN-10747. Bump YARN CSI protobuf version to 3.7.1

2021-04-22 Thread GitBox
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

[GitHub] [hadoop] smengcl commented on pull request #2946: YARN-10747. Bump YARN CSI protobuf version to 3.7.1

2021-04-26 Thread GitBox
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

[GitHub] [hadoop] smengcl commented on pull request #2946: YARN-10747. Bump YARN CSI protobuf version to 3.7.1

2021-04-28 Thread GitBox
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