You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by ma...@apache.org on 2016/07/26 05:36:42 UTC

[2/2] kylin git commit: remove log in CubeSegment

remove log in CubeSegment


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

Branch: refs/heads/master
Commit: 467226f409bf646cea1329e3638f8cfa28fee885
Parents: 49b37ef
Author: Hongbin Ma <ma...@apache.org>
Authored: Mon Jul 25 16:07:31 2016 +0800
Committer: Hongbin Ma <ma...@apache.org>
Committed: Tue Jul 26 13:20:02 2016 +0800

----------------------------------------------------------------------
 .../src/main/java/org/apache/kylin/cube/CubeSegment.java | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kylin/blob/467226f4/core-cube/src/main/java/org/apache/kylin/cube/CubeSegment.java
----------------------------------------------------------------------
diff --git a/core-cube/src/main/java/org/apache/kylin/cube/CubeSegment.java b/core-cube/src/main/java/org/apache/kylin/cube/CubeSegment.java
index febfa86..abfb5ff 100644
--- a/core-cube/src/main/java/org/apache/kylin/cube/CubeSegment.java
+++ b/core-cube/src/main/java/org/apache/kylin/cube/CubeSegment.java
@@ -41,8 +41,6 @@ import org.apache.kylin.metadata.model.IJoinedFlatTableDesc;
 import org.apache.kylin.metadata.model.SegmentStatusEnum;
 import org.apache.kylin.metadata.model.TblColRef;
 import org.apache.kylin.metadata.realization.IRealization;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 import com.fasterxml.jackson.annotation.JsonAutoDetect;
 import com.fasterxml.jackson.annotation.JsonBackReference;
@@ -54,7 +52,6 @@ import com.google.common.collect.Maps;
 
 @JsonAutoDetect(fieldVisibility = Visibility.NONE, getterVisibility = Visibility.NONE, isGetterVisibility = Visibility.NONE, setterVisibility = Visibility.NONE)
 public class CubeSegment implements Comparable<CubeSegment>, IBuildable {
-    private static final Logger logger = LoggerFactory.getLogger(CubeSegment.class);
 
     @JsonBackReference
     private CubeInstance cubeInstance;
@@ -495,11 +492,11 @@ public class CubeSegment implements Comparable<CubeSegment>, IBuildable {
     public int getTotalShards(long cuboidId) {
         if (totalShards > 0) {
             //shard squashed case
-            logger.info("total shards for {} is {}", cuboidId, totalShards);
+            //logger.info("total shards for {} is {}", cuboidId, totalShards);
             return totalShards;
         } else {
             int ret = getCuboidShardNum(cuboidId);
-            logger.info("total shards for {} is {}", cuboidId, ret);
+            //logger.info("total shards for {} is {}", cuboidId, ret);
             return ret;
         }
     }
@@ -518,10 +515,10 @@ public class CubeSegment implements Comparable<CubeSegment>, IBuildable {
                 cuboidBaseShards.put(cuboidId, ret);
             }
 
-            logger.info("base for cuboid {} is {}", cuboidId, ret);
+            //logger.info("base for cuboid {} is {}", cuboidId, ret);
             return ret;
         } else {
-            logger.info("base for cuboid {} is {}", cuboidId, 0);
+            //logger.info("base for cuboid {} is {}", cuboidId, 0);
             return 0;
         }
     }