Repository: kylin
Updated Branches:
  refs/heads/yang-m1 29b3914a2 -> b11491549


minor, add commit to htable


Project: http://git-wip-us.apache.org/repos/asf/kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/b1149154
Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/b1149154
Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/b1149154

Branch: refs/heads/yang-m1
Commit: b11491549dd2c4f1b51cf9aee43bc7d9a9ee4dc1
Parents: 29b3914
Author: lidongsjtu <lid...@apache.org>
Authored: Thu Apr 14 16:16:48 2016 +0800
Committer: lidongsjtu <lid...@apache.org>
Committed: Thu Apr 14 16:16:57 2016 +0800

----------------------------------------------------------------------
 .../src/main/java/org/apache/kylin/common/KylinVersion.java | 1 -
 .../kylin/metadata/realization/IRealizationConstants.java   | 2 ++
 .../apache/kylin/storage/hbase/steps/CubeHTableUtil.java    | 6 ++++++
 .../kylin/storage/hbase/util/DeployCoprocessorCLI.java      | 9 +++++++++
 4 files changed, 17 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kylin/blob/b1149154/core-common/src/main/java/org/apache/kylin/common/KylinVersion.java
----------------------------------------------------------------------
diff --git 
a/core-common/src/main/java/org/apache/kylin/common/KylinVersion.java 
b/core-common/src/main/java/org/apache/kylin/common/KylinVersion.java
index 91c1958..ccbdd54 100644
--- a/core-common/src/main/java/org/apache/kylin/common/KylinVersion.java
+++ b/core-common/src/main/java/org/apache/kylin/common/KylinVersion.java
@@ -23,7 +23,6 @@ import java.util.Set;
 
 import javax.annotation.Nullable;
 
-import org.apache.commons.collections.CollectionUtils;
 import org.apache.commons.io.FileUtils;
 import org.apache.commons.lang.StringUtils;
 

http://git-wip-us.apache.org/repos/asf/kylin/blob/b1149154/core-metadata/src/main/java/org/apache/kylin/metadata/realization/IRealizationConstants.java
----------------------------------------------------------------------
diff --git 
a/core-metadata/src/main/java/org/apache/kylin/metadata/realization/IRealizationConstants.java
 
b/core-metadata/src/main/java/org/apache/kylin/metadata/realization/IRealizationConstants.java
index 138a209..efb5e36 100644
--- 
a/core-metadata/src/main/java/org/apache/kylin/metadata/realization/IRealizationConstants.java
+++ 
b/core-metadata/src/main/java/org/apache/kylin/metadata/realization/IRealizationConstants.java
@@ -38,4 +38,6 @@ public class IRealizationConstants {
 
     public final static String HTableCreationTime = "CREATION_TIME";
 
+    public final static String HTableGitTag = "GIT_COMMIT";
+
 }

http://git-wip-us.apache.org/repos/asf/kylin/blob/b1149154/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/steps/CubeHTableUtil.java
----------------------------------------------------------------------
diff --git 
a/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/steps/CubeHTableUtil.java
 
b/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/steps/CubeHTableUtil.java
index aed28ad..67c8518 100644
--- 
a/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/steps/CubeHTableUtil.java
+++ 
b/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/steps/CubeHTableUtil.java
@@ -33,6 +33,7 @@ import org.apache.hadoop.hbase.regionserver.BloomType;
 import org.apache.hadoop.hbase.regionserver.DisabledRegionSplitPolicy;
 import org.apache.hadoop.hbase.security.User;
 import org.apache.kylin.common.KylinConfig;
+import org.apache.kylin.common.KylinVersion;
 import org.apache.kylin.cube.CubeInstance;
 import org.apache.kylin.cube.CubeSegment;
 import org.apache.kylin.cube.model.CubeDesc;
@@ -67,6 +68,11 @@ public class CubeHTableUtil {
             tableDesc.setValue(IRealizationConstants.HTableOwner, 
kylinConfig.getKylinOwner());
         }
 
+        String commitInfo = KylinVersion.getGitCommitInfo();
+        if (!StringUtils.isEmpty(commitInfo)) {
+            tableDesc.setValue(IRealizationConstants.HTableGitTag, commitInfo);
+        }
+
         //HTableUser is the cube owner, which will be the "user"
         tableDesc.setValue(IRealizationConstants.HTableUser, 
cubeInstance.getOwner());
 

http://git-wip-us.apache.org/repos/asf/kylin/blob/b1149154/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/util/DeployCoprocessorCLI.java
----------------------------------------------------------------------
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 8bf06b7..2a5c950 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
@@ -43,6 +43,7 @@ import org.apache.hadoop.hbase.TableNotFoundException;
 import org.apache.hadoop.hbase.client.HBaseAdmin;
 import org.apache.hadoop.hbase.io.ImmutableBytesWritable;
 import org.apache.kylin.common.KylinConfig;
+import org.apache.kylin.common.KylinVersion;
 import org.apache.kylin.common.util.Bytes;
 import org.apache.kylin.cube.CubeInstance;
 import org.apache.kylin.cube.CubeManager;
@@ -51,6 +52,7 @@ import org.apache.kylin.invertedindex.IIInstance;
 import org.apache.kylin.invertedindex.IIManager;
 import org.apache.kylin.invertedindex.IISegment;
 import org.apache.kylin.metadata.model.SegmentStatusEnum;
+import org.apache.kylin.metadata.realization.IRealizationConstants;
 import org.apache.kylin.metadata.realization.RealizationStatusEnum;
 import org.apache.kylin.storage.hbase.HBaseConnection;
 import org.slf4j.Logger;
@@ -210,6 +212,13 @@ public class DeployCoprocessorCLI {
             desc.removeCoprocessor(IIEndpointClassOld);
         }
         addCoprocessorOnHTable(desc, hdfsCoprocessorJar);
+
+        // update commit tags
+        String commitInfo = KylinVersion.getGitCommitInfo();
+        if (!StringUtils.isEmpty(commitInfo)) {
+            desc.setValue(IRealizationConstants.HTableGitTag, commitInfo);
+        }
+
         hbaseAdmin.modifyTable(tableName, desc);
 
         logger.info("Enable " + tableName);

Reply via email to