KYLIN-2333 Kylin doesn't need 0-D cuboid, can remove that step
Project: http://git-wip-us.apache.org/repos/asf/kylin/repo Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/687d5fd1 Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/687d5fd1 Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/687d5fd1 Branch: refs/heads/master-hbase1.x Commit: 687d5fd150b6019e71db0f600307bbd4f7c749f9 Parents: 1a190ec Author: shaofengshi <shaofeng...@apache.org> Authored: Thu Dec 29 13:23:14 2016 +0800 Committer: shaofengshi <shaofeng...@apache.org> Committed: Thu Dec 29 13:23:14 2016 +0800 ---------------------------------------------------------------------- .../java/org/apache/kylin/engine/mr/BatchCubingJobBuilder2.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/kylin/blob/687d5fd1/engine-mr/src/main/java/org/apache/kylin/engine/mr/BatchCubingJobBuilder2.java ---------------------------------------------------------------------- diff --git a/engine-mr/src/main/java/org/apache/kylin/engine/mr/BatchCubingJobBuilder2.java b/engine-mr/src/main/java/org/apache/kylin/engine/mr/BatchCubingJobBuilder2.java index 6eba3c2..966bb1b 100644 --- a/engine-mr/src/main/java/org/apache/kylin/engine/mr/BatchCubingJobBuilder2.java +++ b/engine-mr/src/main/java/org/apache/kylin/engine/mr/BatchCubingJobBuilder2.java @@ -84,7 +84,7 @@ public class BatchCubingJobBuilder2 extends JobBuilderSupport { // base cuboid step result.addTask(createBaseCuboidStep(cuboidOutputTempPath, jobId)); // n dim cuboid steps - for (int i = 1; i <= groupRowkeyColumnsCount; i++) { + for (int i = 1; i < groupRowkeyColumnsCount; i++) { int dimNum = totalRowkeyColumnsCount - i; result.addTask(createNDimensionCuboidStep(cuboidOutputTempPath, dimNum, totalRowkeyColumnsCount, jobId)); }
