[kylin] branch 2.6.x updated: KYLIN-3911: Check if HBase table is enabled before diabling table in DeployCoprocessorCLI

[ni...@apache.org]

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

nic pushed a commit to branch 2.6.x
in repository https://gitbox.apache.org/repos/asf/kylin.git


The following commit(s) were added to refs/heads/2.6.x by this push:
     new b09815c  KYLIN-3911: Check if HBase table is enabled before diabling table in DeployCoprocessorCLI
b09815c is described below

commit b09815c17be7eeb70ec48b3973518ae3e3d30217
Author: Liu Shaohui <li...@xiaomi.com>
AuthorDate: Wed Mar 27 13:01:12 2019 +0800

    KYLIN-3911: Check if HBase table is enabled before diabling table in DeployCoprocessorCLI
---
 .../org/apache/kylin/storage/hbase/util/DeployCoprocessorCLI.java     | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/util/DeployCoprocessorCLI.java b/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/util/DeployCoprocessorCLI.java
index 0af00ef..e3dac79 100644
--- a/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/util/DeployCoprocessorCLI.java
+++ b/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/util/DeployCoprocessorCLI.java
@@ -294,7 +294,9 @@ public class DeployCoprocessorCLI {
         logger.info("reset coprocessor on " + tableName);
 
         logger.info("Disable " + tableName);
-        hbaseAdmin.disableTable(TableName.valueOf(tableName));
+        if (hbaseAdmin.isTableEnabled(TableName.valueOf(tableName))) {
+            hbaseAdmin.disableTable(TableName.valueOf(tableName));
+        }
 
         while (desc.hasCoprocessor(CubeObserverClassOld2)) {
             desc.removeCoprocessor(CubeObserverClassOld2);