You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by zh...@apache.org on 2020/08/12 02:03:55 UTC

[hbase] branch branch-2 updated: HBASE-24856 Fix error prone error in FlushTableSubprocedure (#2242)

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

zhangduo pushed a commit to branch branch-2
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-2 by this push:
     new 96ea136  HBASE-24856 Fix error prone error in FlushTableSubprocedure (#2242)
96ea136 is described below

commit 96ea136a8ede3a3c919e7255a52360fa0af18ce8
Author: Duo Zhang <zh...@apache.org>
AuthorDate: Wed Aug 12 09:51:12 2020 +0800

    HBASE-24856 Fix error prone error in FlushTableSubprocedure (#2242)
    
    Signed-off-by: Viraj Jasani <vj...@apache.org>
    Signed-off-by: Jan Hentschel <ja...@ultratendency.com>
---
 .../apache/hadoop/hbase/procedure/flush/FlushTableSubprocedure.java    | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/procedure/flush/FlushTableSubprocedure.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/procedure/flush/FlushTableSubprocedure.java
index d124039..f506d53 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/procedure/flush/FlushTableSubprocedure.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/procedure/flush/FlushTableSubprocedure.java
@@ -18,6 +18,7 @@
 package org.apache.hadoop.hbase.procedure.flush;
 
 import java.util.Arrays;
+import java.util.Collections;
 import java.util.List;
 import java.util.concurrent.Callable;
 
@@ -102,7 +103,7 @@ public class FlushTableSubprocedure extends Subprocedure {
     List<byte[]> families = null;
     if (family != null) {
       LOG.debug("About to flush family {} on all regions for table {}", family, table);
-      families = Arrays.asList(Bytes.toBytes(family));
+      families = Collections.singletonList(Bytes.toBytes(family));
     }
     // Add all hfiles already existing in region.
     for (HRegion region : regions) {