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

[shaofengshi]

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

shaofengshi pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kylin.git


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

commit 59378efca9b4ebb865bc90709e292d3e7b412ebe
Author: Liu Shaohui <liushao...@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 853882e..6318245 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);