From a99676e39b381d8f5616fab0cea733ec6cc6ae3b Mon Sep 17 00:00:00 2001
From: Hao Chen <chenh1024@gmail.com>
Date: Thu, 21 Feb 2019 04:37:04 +0800
Subject: [PATCH] [Java] lint unused imports (#4100)

---
 .../main/java/org/ray/api/options/ActorCreationOptions.java    | 1 -
 java/api/src/main/java/org/ray/api/runtime/RayRuntime.java     | 1 -
 java/checkstyle.xml                                            | 3 +++
 .../src/main/java/org/ray/runtime/AbstractRayRuntime.java      | 2 --
 .../java/org/ray/runtime/objectstore/ObjectStoreProxy.java     | 2 --
 java/test/src/main/java/org/ray/api/test/BaseTest.java         | 1 -
 .../main/java/org/ray/api/test/MultiLanguageClusterTest.java   | 1 -
 java/test/src/main/java/org/ray/api/test/WordCountTest.java    | 1 -
 8 files changed, 3 insertions(+), 9 deletions(-)

diff --git a/java/api/src/main/java/org/ray/api/options/ActorCreationOptions.java b/java/api/src/main/java/org/ray/api/options/ActorCreationOptions.java
index 267b7a354..e4f54f009 100644
--- a/java/api/src/main/java/org/ray/api/options/ActorCreationOptions.java
+++ b/java/api/src/main/java/org/ray/api/options/ActorCreationOptions.java
@@ -1,6 +1,5 @@
 package org.ray.api.options;
 
-import java.util.HashMap;
 import java.util.Map;
 
 /**
diff --git a/java/api/src/main/java/org/ray/api/runtime/RayRuntime.java b/java/api/src/main/java/org/ray/api/runtime/RayRuntime.java
index 242190802..48ebd6fce 100644
--- a/java/api/src/main/java/org/ray/api/runtime/RayRuntime.java
+++ b/java/api/src/main/java/org/ray/api/runtime/RayRuntime.java
@@ -7,7 +7,6 @@ import org.ray.api.WaitResult;
 import org.ray.api.function.RayFunc;
 import org.ray.api.id.UniqueId;
 import org.ray.api.options.ActorCreationOptions;
-import org.ray.api.options.BaseTaskOptions;
 import org.ray.api.options.CallOptions;
 
 /**
diff --git a/java/checkstyle.xml b/java/checkstyle.xml
index 88f970e04..ef13e373c 100644
--- a/java/checkstyle.xml
+++ b/java/checkstyle.xml
@@ -28,6 +28,9 @@
     </module>
 
     <module name="TreeWalker">
+        <!-- Below are custom config items in addition to the original google style config. -->
+        <module name="UnusedImports"/>
+        <!-- End of custom config items -->
         <module name="OuterTypeFilename"/>
         <module name="IllegalTokenText">
             <property name="tokens" value="STRING_LITERAL, CHAR_LITERAL"/>
diff --git a/java/runtime/src/main/java/org/ray/runtime/AbstractRayRuntime.java b/java/runtime/src/main/java/org/ray/runtime/AbstractRayRuntime.java
index 2ad1028f8..fe2f2b866 100644
--- a/java/runtime/src/main/java/org/ray/runtime/AbstractRayRuntime.java
+++ b/java/runtime/src/main/java/org/ray/runtime/AbstractRayRuntime.java
@@ -1,7 +1,5 @@
 package org.ray.runtime;
 
-import static java.util.stream.Collectors.toList;
-
 import com.google.common.collect.ImmutableList;
 import java.util.ArrayList;
 import java.util.Arrays;
diff --git a/java/runtime/src/main/java/org/ray/runtime/objectstore/ObjectStoreProxy.java b/java/runtime/src/main/java/org/ray/runtime/objectstore/ObjectStoreProxy.java
index 448c05364..e3d8f2e58 100644
--- a/java/runtime/src/main/java/org/ray/runtime/objectstore/ObjectStoreProxy.java
+++ b/java/runtime/src/main/java/org/ray/runtime/objectstore/ObjectStoreProxy.java
@@ -8,10 +8,8 @@ import org.apache.arrow.plasma.ObjectStoreLink;
 import org.apache.arrow.plasma.ObjectStoreLink.ObjectStoreData;
 import org.apache.arrow.plasma.PlasmaClient;
 import org.apache.arrow.plasma.exceptions.DuplicateObjectException;
-import org.apache.commons.lang3.tuple.Pair;
 import org.ray.api.exception.RayActorException;
 import org.ray.api.exception.RayException;
-import org.ray.api.exception.RayTaskException;
 import org.ray.api.exception.RayWorkerException;
 import org.ray.api.exception.UnreconstructableException;
 import org.ray.api.id.UniqueId;
diff --git a/java/test/src/main/java/org/ray/api/test/BaseTest.java b/java/test/src/main/java/org/ray/api/test/BaseTest.java
index 22c35670a..0f73e2d1c 100644
--- a/java/test/src/main/java/org/ray/api/test/BaseTest.java
+++ b/java/test/src/main/java/org/ray/api/test/BaseTest.java
@@ -1,6 +1,5 @@
 package org.ray.api.test;
 
-import java.io.File;
 import org.ray.api.Ray;
 import org.testng.annotations.AfterMethod;
 import org.testng.annotations.BeforeMethod;
diff --git a/java/test/src/main/java/org/ray/api/test/MultiLanguageClusterTest.java b/java/test/src/main/java/org/ray/api/test/MultiLanguageClusterTest.java
index d487acc03..b3a8e87b7 100644
--- a/java/test/src/main/java/org/ray/api/test/MultiLanguageClusterTest.java
+++ b/java/test/src/main/java/org/ray/api/test/MultiLanguageClusterTest.java
@@ -2,7 +2,6 @@ package org.ray.api.test;
 
 import com.google.common.collect.ImmutableList;
 import java.io.File;
-import java.io.IOException;
 import java.lang.ProcessBuilder.Redirect;
 import java.util.List;
 import java.util.concurrent.TimeUnit;
diff --git a/java/test/src/main/java/org/ray/api/test/WordCountTest.java b/java/test/src/main/java/org/ray/api/test/WordCountTest.java
index 43a1ae7cd..6efedf8e2 100644
--- a/java/test/src/main/java/org/ray/api/test/WordCountTest.java
+++ b/java/test/src/main/java/org/ray/api/test/WordCountTest.java
@@ -8,7 +8,6 @@ import org.ray.api.RayObject;
 import org.ray.api.annotation.RayRemote;
 import org.ray.runtime.util.FileUtil;
 import org.testng.Assert;
-import org.testng.annotations.Test;
 
 /**
  * given a directory of document files on each "machine", we would like to count the appearance of