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);