Zoltan Chovan has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/24518 )
Change subject: [java] Replace deprecated APIs to silence build warnings
......................................................................
[java] Replace deprecated APIs to silence build warnings
Gradle 9 / newer toolchains surface a number of deprecation warnings.
In this change the deprecated APIs are replaced with their supported
equivalents:
- org.junit.Assert.assertThat -> org.hamcrest.MatcherAssert.assertThat
(deprecated in JUnit 4.13) across the client, hive, and security
tests.
- commons-cli BasicParser -> DefaultParser in the kudu-subprocess config.
- new Integer(...) -> Integer.valueOf(...) in kudu-backup TableMetadata.
- Guava Throwables.propagateIfPossible -> explicit instanceof rethrow
in FakeDNS, and drop the now-unused import.
No behavioral change; this only removes warning output from the build.
Change-Id: Ia10fdbca3733d9b16caf8d4c0ad00e2234092d39
Reviewed-on: http://gerrit.cloudera.org:8080/24518
Reviewed-by: Marton Greber <[email protected]>
Tested-by: Marton Greber <[email protected]>
Reviewed-by: Gabriella Lotz <[email protected]>
---
M
java/kudu-backup-common/src/main/scala/org/apache/kudu/backup/TableMetadata.scala
M
java/kudu-client/src/test/java/org/apache/kudu/client/TestConnectToCluster.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduClientBasic.java
M
java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduClientSecurity.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduScanner.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduSession.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduTable.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestPartitionPruner.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurity.java
M
java/kudu-hive/src/test/java/org/apache/kudu/hive/metastore/TestKuduMetastorePlugin.java
M
java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/DefaultSourceTest.scala
M
java/kudu-subprocess/src/main/java/org/apache/kudu/subprocess/SubprocessConfiguration.java
M
java/kudu-subprocess/src/main/java/org/apache/kudu/subprocess/ranger/authorization/RangerKuduAuthorizer.java
M java/kudu-test-utils/src/main/java/org/apache/kudu/test/cluster/FakeDNS.java
14 files changed, 67 insertions(+), 33 deletions(-)
Approvals:
Marton Greber: Looks good to me, approved; Verified
Gabriella Lotz: Looks good to me, but someone else must approve
--
To view, visit http://gerrit.cloudera.org:8080/24518
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia10fdbca3733d9b16caf8d4c0ad00e2234092d39
Gerrit-Change-Number: 24518
Gerrit-PatchSet: 8
Gerrit-Owner: Zoltan Chovan <[email protected]>
Gerrit-Reviewer: Gabriella Lotz <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>
Gerrit-Reviewer: Zoltan Chovan <[email protected]>
