Repository: kylin
Updated Branches:
  refs/heads/master 48e164924 -> edab8d03a


minor, more robus DoggedCubeBuilder in terms of mem reservation


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

Branch: refs/heads/master
Commit: edab8d03a21da4ef6637f0b74568da0286e6746c
Parents: 48e1649
Author: Li Yang <liy...@apache.org>
Authored: Wed Jun 22 17:06:01 2016 +0800
Committer: Li Yang <liy...@apache.org>
Committed: Wed Jun 22 17:06:01 2016 +0800

----------------------------------------------------------------------
 .../org/apache/kylin/cube/inmemcubing/DoggedCubeBuilder.java     | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kylin/blob/edab8d03/core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/DoggedCubeBuilder.java
----------------------------------------------------------------------
diff --git 
a/core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/DoggedCubeBuilder.java
 
b/core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/DoggedCubeBuilder.java
index af6ef82..606c820 100644
--- 
a/core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/DoggedCubeBuilder.java
+++ 
b/core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/DoggedCubeBuilder.java
@@ -252,8 +252,8 @@ public class DoggedCubeBuilder extends 
AbstractInMemCubeBuilder {
                 return true;
             }
 
-            if (systemAvailMB <= reserveMemoryMB) {
-                logger.info("Split cut due to hitting memory threshold, system 
avail " + systemAvailMB + " MB <= reserve " + reserveMemoryMB + " MB");
+            if (systemAvailMB <= reserveMemoryMB * 1.5) {
+                logger.info("Split cut due to hitting memory threshold, system 
avail " + systemAvailMB + " MB <= reserve " + reserveMemoryMB + "*1.5 MB");
                 return true;
             }
 

Reply via email to