This is an automated email from the ASF dual-hosted git repository.

nishant pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-druid.git


The following commit(s) were added to refs/heads/master by this push:
     new e8dedb3  Enable MS_MUTABLE_COLLECTION_PKGPROTECT (#7971)
e8dedb3 is described below

commit e8dedb3de4e026b122357b3bbe26325271d03467
Author: Fokko Driesprong <[email protected]>
AuthorDate: Mon Jul 1 07:54:05 2019 +0200

    Enable MS_MUTABLE_COLLECTION_PKGPROTECT (#7971)
---
 codestyle/spotbugs-exclude.xml                                         | 1 -
 .../java/org/apache/druid/indexing/common/task/HadoopIndexTask.java    | 3 +--
 2 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/codestyle/spotbugs-exclude.xml b/codestyle/spotbugs-exclude.xml
index 60a65c6..639fab7 100644
--- a/codestyle/spotbugs-exclude.xml
+++ b/codestyle/spotbugs-exclude.xml
@@ -55,7 +55,6 @@
     <Bug pattern="JLM_JSR166_UTILCONCURRENT_MONITORENTER"/>
     <Bug pattern="JLM_JSR166_UTILCONCURRENT_MONITORENTER"/>
     <Bug pattern="MS_FINAL_PKGPROTECT"/>
-    <Bug pattern="MS_MUTABLE_COLLECTION_PKGPROTECT"/>
     <Bug pattern="MS_OOI_PKGPROTECT"/>
     <Bug pattern="MS_PKGPROTECT"/>
     <Bug pattern="NP_EQUALS_SHOULD_HANDLE_NULL_ARGUMENT"/>
diff --git 
a/indexing-service/src/main/java/org/apache/druid/indexing/common/task/HadoopIndexTask.java
 
b/indexing-service/src/main/java/org/apache/druid/indexing/common/task/HadoopIndexTask.java
index 5b67c25..2571c57 100644
--- 
a/indexing-service/src/main/java/org/apache/druid/indexing/common/task/HadoopIndexTask.java
+++ 
b/indexing-service/src/main/java/org/apache/druid/indexing/common/task/HadoopIndexTask.java
@@ -75,7 +75,6 @@ import javax.ws.rs.core.Response;
 import java.io.File;
 import java.lang.reflect.InvocationTargetException;
 import java.lang.reflect.Method;
-import java.util.Arrays;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
@@ -528,7 +527,7 @@ public class HadoopIndexTask extends HadoopTask implements 
ChatHandler
 
   public static class InnerProcessingStatsGetter implements TaskMetricsGetter
   {
-    public static final List<String> KEYS = Arrays.asList(
+    static final List<String> KEYS = ImmutableList.of(
         TaskMetricsUtils.ROWS_PROCESSED,
         TaskMetricsUtils.ROWS_PROCESSED_WITH_ERRORS,
         TaskMetricsUtils.ROWS_THROWN_AWAY,


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to