Skip to content

Commit 4331c88

Browse files
authored
HADOOP-17344. Harmonize guava version and shade guava in yarn-csi. (#2434)
Reviewed-by: Ayush Saxena <ayushsaxena@apache.org> Reviewed-by: Wei-Chiu Chuang <weichiu@apache.org>
1 parent 0946f58 commit 4331c88

File tree

4 files changed

+7
-21
lines changed

4 files changed

+7
-21
lines changed

hadoop-yarn-project/hadoop-yarn/hadoop-yarn-csi/pom.xml

Lines changed: 2 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -27,16 +27,14 @@
2727

2828
<properties>
2929
<protobuf.version>3.6.1</protobuf.version>
30-
<guava.version>20.0</guava.version>
3130
<grpc.version>1.26.0</grpc.version>
3231
<os-maven-plugin.version>1.5.0.Final</os-maven-plugin.version>
3332
</properties>
3433

3534
<dependencies>
3635
<dependency>
37-
<groupId>com.google.guava</groupId>
38-
<artifactId>guava</artifactId>
39-
<version>${guava.version}</version>
36+
<groupId>org.apache.hadoop.thirdparty</groupId>
37+
<artifactId>hadoop-shaded-guava</artifactId>
4038
</dependency>
4139
<dependency>
4240
<groupId>com.google.protobuf</groupId>
@@ -209,18 +207,6 @@
209207
</execution>
210208
</executions>
211209
</plugin>
212-
<plugin>
213-
<groupId>com.google.code.maven-replacer-plugin</groupId>
214-
<artifactId>replacer</artifactId>
215-
<executions>
216-
<execution>
217-
<id>replace-guava</id>
218-
<configuration>
219-
<skip>true</skip>
220-
</configuration>
221-
</execution>
222-
</executions>
223-
</plugin>
224210
</plugins>
225211
</build>
226212
</project>

hadoop-yarn-project/hadoop-yarn/hadoop-yarn-csi/src/test/java/org/apache/hadoop/yarn/csi/adaptor/TestCsiAdaptorService.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,15 +17,15 @@
1717
*/
1818
package org.apache.hadoop.yarn.csi.adaptor;
1919

20-
import com.google.common.collect.ImmutableList;
21-
import com.google.common.collect.ImmutableMap;
2220
import csi.v0.Csi;
2321
import org.apache.commons.io.FileUtils;
2422
import org.apache.hadoop.conf.Configuration;
2523
import org.apache.hadoop.net.NetUtils;
2624
import org.apache.hadoop.security.UserGroupInformation;
2725
import org.apache.hadoop.service.ServiceStateException;
2826
import org.apache.hadoop.test.GenericTestUtils;
27+
import org.apache.hadoop.thirdparty.com.google.common.collect.ImmutableList;
28+
import org.apache.hadoop.thirdparty.com.google.common.collect.ImmutableMap;
2929
import org.apache.hadoop.yarn.api.CsiAdaptorProtocol;
3030
import org.apache.hadoop.yarn.api.CsiAdaptorPlugin;
3131
import org.apache.hadoop.yarn.api.impl.pb.client.CsiAdaptorProtocolPBClientImpl;

hadoop-yarn-project/hadoop-yarn/hadoop-yarn-csi/src/test/java/org/apache/hadoop/yarn/csi/adaptor/TestValidateVolumeCapabilityRequest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,8 @@
1717
*/
1818
package org.apache.hadoop.yarn.csi.adaptor;
1919

20-
import com.google.common.collect.ImmutableList;
21-
import com.google.common.collect.ImmutableMap;
20+
import org.apache.hadoop.thirdparty.com.google.common.collect.ImmutableList;
21+
import org.apache.hadoop.thirdparty.com.google.common.collect.ImmutableMap;
2222
import org.apache.hadoop.yarn.api.protocolrecords.ValidateVolumeCapabilitiesRequest;
2323
import org.apache.hadoop.yarn.api.protocolrecords.ValidateVolumeCapabilitiesRequest.VolumeCapability;
2424
import org.apache.hadoop.yarn.api.protocolrecords.impl.pb.ValidateVolumeCapabilitiesRequestPBImpl;

hadoop-yarn-project/hadoop-yarn/hadoop-yarn-csi/src/test/java/org/apache/hadoop/yarn/csi/client/TestCsiClient.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020

2121
import csi.v0.Csi;
2222
import org.apache.commons.io.FileUtils;
23-
import com.google.common.io.Files;
23+
import org.apache.hadoop.thirdparty.com.google.common.io.Files;
2424
import org.junit.AfterClass;
2525
import org.junit.Assert;
2626
import org.junit.Assume;

0 commit comments

Comments
 (0)