kylin git commit: KYLIN-2688 When the model has a ready cube, should not allow user to edit model JSON in web.

2017-06-29 Thread chen
Repository: kylin
Updated Branches:
  refs/heads/master d851c51d2 -> a8627e8a9


KYLIN-2688 When the model has a ready cube, should not allow user to edit model 
JSON in web.


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

Branch: refs/heads/master
Commit: a8627e8a9f264cc138124f3825d490463fa58480
Parents: d851c51
Author: 10069681 
Authored: Tue Jun 27 15:09:56 2017 +0800
Committer: chenzhx <346839...@qq.com>
Committed: Thu Jun 29 19:16:02 2017 +0800

--
 webapp/app/js/controllers/models.js | 13 -
 webapp/app/partials/models/models_tree.html |  4 ++--
 2 files changed, 10 insertions(+), 7 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/kylin/blob/a8627e8a/webapp/app/js/controllers/models.js
--
diff --git a/webapp/app/js/controllers/models.js 
b/webapp/app/js/controllers/models.js
index fd21845..1cc0c84 100644
--- a/webapp/app/js/controllers/models.js
+++ b/webapp/app/js/controllers/models.js
@@ -127,7 +127,7 @@ KylinApp.controller('ModelsCtrl', function ($scope, $q, 
$routeParams, $location,
 });
   };
 
-  $scope.editModel = function(model){
+  $scope.editModel = function(model, isEditJson){
 var cubename = [];
 var modelstate=false;
 var i=0;
@@ -145,10 +145,13 @@ KylinApp.controller('ModelsCtrl', function ($scope, $q, 
$routeParams, $location,
 })
   }
 
-  if(modelstate==false){
-$location.path("/models/edit/"+model.name);
-  }
-  else{
+  if (modelstate==false){
+ if (isEditJson) {
+ $location.path("/models/edit/" + model.name + 
"/descriptionjson");
+ } else {
+ $location.path("/models/edit/" + model.name);
+ }
+  } else {
 SweetAlert.swal('Sorry','This model is still used by '+ 
cubename.join(','));
   }
 })

http://git-wip-us.apache.org/repos/asf/kylin/blob/a8627e8a/webapp/app/partials/models/models_tree.html
--
diff --git a/webapp/app/partials/models/models_tree.html 
b/webapp/app/partials/models/models_tree.html
index c7b55a3..399f839 100644
--- a/webapp/app/partials/models/models_tree.html
+++ b/webapp/app/partials/models/models_tree.html
@@ -53,11 +53,11 @@
   Action 
 
 
-  Edit
+  Edit
   Clone 
   Drop
   
-Edit(JSON)
+Edit(JSON)
 
   
 



kylin git commit: KYLIN-2695 Should allow user to override spark conf in cube

2017-06-29 Thread kangkaisen
Repository: kylin
Updated Branches:
  refs/heads/master a8627e8a9 -> 804a1aa38


KYLIN-2695 Should allow user to override spark conf in cube


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

Branch: refs/heads/master
Commit: 804a1aa3809e67d8e701f147bee794eec19d57e2
Parents: a8627e8
Author: kangkaisen 
Authored: Wed Jun 28 20:21:52 2017 +0800
Committer: kangkaisen 
Committed: Thu Jun 29 21:03:20 2017 +0800

--
 .../java/org/apache/kylin/engine/spark/SparkExecutable.java   | 7 ++-
 1 file changed, 6 insertions(+), 1 deletion(-)
--


http://git-wip-us.apache.org/repos/asf/kylin/blob/804a1aa3/engine-spark/src/main/java/org/apache/kylin/engine/spark/SparkExecutable.java
--
diff --git 
a/engine-spark/src/main/java/org/apache/kylin/engine/spark/SparkExecutable.java 
b/engine-spark/src/main/java/org/apache/kylin/engine/spark/SparkExecutable.java
index 1ed2235..1e032c6 100644
--- 
a/engine-spark/src/main/java/org/apache/kylin/engine/spark/SparkExecutable.java
+++ 
b/engine-spark/src/main/java/org/apache/kylin/engine/spark/SparkExecutable.java
@@ -23,6 +23,8 @@ import java.util.Map;
 import org.apache.commons.lang.StringUtils;
 import org.apache.kylin.common.KylinConfig;
 import org.apache.kylin.common.util.CliCommandExecutor;
+import org.apache.kylin.cube.CubeInstance;
+import org.apache.kylin.cube.CubeManager;
 import org.apache.kylin.job.common.PatternedLogger;
 import org.apache.kylin.job.exception.ExecuteException;
 import org.apache.kylin.job.execution.AbstractExecutable;
@@ -70,7 +72,10 @@ public class SparkExecutable extends AbstractExecutable {
 
 @Override
 protected ExecuteResult doWork(ExecutableContext context) throws 
ExecuteException {
-final KylinConfig config = context.getConfig();
+String cubeName = 
this.getParam(SparkCubingByLayer.OPTION_CUBE_NAME.getOpt());
+CubeInstance cube = 
CubeManager.getInstance(context.getConfig()).getCube(cubeName);
+final KylinConfig config = cube.getConfig();
+
 if (KylinConfig.getSparkHome() == null) {
 throw new NullPointerException();
 }



kylin git commit: KYLIN-2694 Fix ArrayIndexOutOfBoundsException in SparkCubingByLayer

2017-06-29 Thread kangkaisen
Repository: kylin
Updated Branches:
  refs/heads/master 804a1aa38 -> 547722d6a


KYLIN-2694 Fix ArrayIndexOutOfBoundsException in SparkCubingByLayer


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

Branch: refs/heads/master
Commit: 547722d6a35344a5675d6d6ce9b885d8fdc58fc7
Parents: 804a1aa
Author: kangkaisen 
Authored: Mon Jun 26 15:33:15 2017 +0800
Committer: kangkaisen 
Committed: Thu Jun 29 21:13:06 2017 +0800

--
 .../java/org/apache/kylin/engine/spark/SparkCubingByLayer.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
--


http://git-wip-us.apache.org/repos/asf/kylin/blob/547722d6/engine-spark/src/main/java/org/apache/kylin/engine/spark/SparkCubingByLayer.java
--
diff --git 
a/engine-spark/src/main/java/org/apache/kylin/engine/spark/SparkCubingByLayer.java
 
b/engine-spark/src/main/java/org/apache/kylin/engine/spark/SparkCubingByLayer.java
index f70fd30..91aa9f7 100644
--- 
a/engine-spark/src/main/java/org/apache/kylin/engine/spark/SparkCubingByLayer.java
+++ 
b/engine-spark/src/main/java/org/apache/kylin/engine/spark/SparkCubingByLayer.java
@@ -269,7 +269,7 @@ public class SparkCubingByLayer extends AbstractApplication 
implements Serializa
 saveToHDFS(allRDDs[level], vCubeDesc.getValue(), outputPath, 
level, confOverwrite);
 allRDDs[level - 1].unpersist();
 }
-allRDDs[totalLevels - 1].unpersist();
+allRDDs[totalLevels].unpersist();
 logger.info("Finished on calculating all level cuboids.");
 }
 



kylin git commit: KYLIN-2693 Should use overrideHiveConfig for LookupHiveViewMaterialization and RedistributeFlatHiveTable

2017-06-29 Thread kangkaisen
Repository: kylin
Updated Branches:
  refs/heads/master 547722d6a -> 772a95f3a


KYLIN-2693 Should use overrideHiveConfig for LookupHiveViewMaterialization and 
RedistributeFlatHiveTable


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

Branch: refs/heads/master
Commit: 772a95f3a0aaafca5260140c461edb053990cfe8
Parents: 547722d
Author: kangkaisen 
Authored: Tue Jun 27 20:02:04 2017 +0800
Committer: kangkaisen 
Committed: Thu Jun 29 21:17:45 2017 +0800

--
 .../src/main/java/org/apache/kylin/source/hive/HiveMRInput.java| 2 ++
 1 file changed, 2 insertions(+)
--


http://git-wip-us.apache.org/repos/asf/kylin/blob/772a95f3/source-hive/src/main/java/org/apache/kylin/source/hive/HiveMRInput.java
--
diff --git 
a/source-hive/src/main/java/org/apache/kylin/source/hive/HiveMRInput.java 
b/source-hive/src/main/java/org/apache/kylin/source/hive/HiveMRInput.java
index 337bafd..4747cb9 100644
--- a/source-hive/src/main/java/org/apache/kylin/source/hive/HiveMRInput.java
+++ b/source-hive/src/main/java/org/apache/kylin/source/hive/HiveMRInput.java
@@ -206,6 +206,7 @@ public class HiveMRInput implements IMRInput {
 }
 
 HiveCmdBuilder hiveCmdBuilder = new HiveCmdBuilder();
+
hiveCmdBuilder.overwriteHiveProps(kylinConfig.getHiveConfigOverride());
 hiveCmdBuilder.addStatement(hiveInitStatements);
 for (TableDesc lookUpTableDesc : lookupViewsTables) {
 String identity = lookUpTableDesc.getIdentity();
@@ -273,6 +274,7 @@ public class HiveMRInput implements IMRInput {
 
 private void redistributeTable(KylinConfig config, int numReducers) 
throws IOException {
 final HiveCmdBuilder hiveCmdBuilder = new HiveCmdBuilder();
+hiveCmdBuilder.overwriteHiveProps(config.getHiveConfigOverride());
 hiveCmdBuilder.addStatement(getInitStatement());
 hiveCmdBuilder.addStatement("set mapreduce.job.reduces=" + 
numReducers + ";\n");
 hiveCmdBuilder.addStatement("set hive.merge.mapredfiles=false;\n");



kylin git commit: KYLIN-2652 Make kylinConfig threadsafe in CubeVisitService

2017-06-29 Thread kangkaisen
Repository: kylin
Updated Branches:
  refs/heads/master 772a95f3a -> a1e055a6b


KYLIN-2652 Make kylinConfig threadsafe in CubeVisitService


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

Branch: refs/heads/master
Commit: a1e055a6b448f7e797ea29bc97e84325b3056638
Parents: 772a95f
Author: kangkaisen 
Authored: Fri May 26 12:30:17 2017 +0800
Committer: kangkaisen 
Committed: Thu Jun 29 21:20:22 2017 +0800

--
 .../hbase/cube/v2/coprocessor/endpoint/CubeVisitService.java  | 7 ++-
 1 file changed, 2 insertions(+), 5 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/kylin/blob/a1e055a6/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/cube/v2/coprocessor/endpoint/CubeVisitService.java
--
diff --git 
a/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/cube/v2/coprocessor/endpoint/CubeVisitService.java
 
b/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/cube/v2/coprocessor/endpoint/CubeVisitService.java
index cde127e..c66af66 100644
--- 
a/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/cube/v2/coprocessor/endpoint/CubeVisitService.java
+++ 
b/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/cube/v2/coprocessor/endpoint/CubeVisitService.java
@@ -238,9 +238,8 @@ public class CubeVisitService extends 
CubeVisitProtos.CubeVisitService implement
 region.startRegionOperation();
 
 // if user change kylin.properties on kylin server, need to 
manually redeploy coprocessor jar to update KylinConfig of Env.
-String serverPropString = request.getKylinProperties();
-KylinConfig.setKylinConfigInEnvIfMissing(serverPropString);
-KylinConfig kylinConfig = KylinConfig.getInstanceFromEnv();
+KylinConfig kylinConfig = 
KylinConfig.createKylinConfig(request.getKylinProperties());
+KylinConfig.setKylinConfigThreadLocal(kylinConfig);
 
 debugGitTag = 
region.getTableDesc().getValue(IRealizationConstants.HTableGitTag);
 
@@ -425,8 +424,6 @@ public class CubeVisitService extends 
CubeVisitProtos.CubeVisitService implement
 
 @Override
 public void stop(CoprocessorEnvironment env) throws IOException {
-// destroy KylinConfig when coprocessor stop
-KylinConfig.destroyInstance();
 }
 
 @Override



kylin git commit: KYLIN-2674 Should not catch OutOfMemoryError in coprocessor

2017-06-29 Thread kangkaisen
Repository: kylin
Updated Branches:
  refs/heads/master a1e055a6b -> 1eb32c3e6


KYLIN-2674 Should not catch OutOfMemoryError in coprocessor


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

Branch: refs/heads/master
Commit: 1eb32c3e62d92142b4f7330ac10edd606c870307
Parents: a1e055a
Author: kangkaisen 
Authored: Mon Jun 12 12:17:20 2017 +0800
Committer: kangkaisen 
Committed: Thu Jun 29 21:23:43 2017 +0800

--
 .../hbase/cube/v2/coprocessor/endpoint/CubeVisitService.java | 4 
 1 file changed, 4 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/kylin/blob/1eb32c3e/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/cube/v2/coprocessor/endpoint/CubeVisitService.java
--
diff --git 
a/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/cube/v2/coprocessor/endpoint/CubeVisitService.java
 
b/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/cube/v2/coprocessor/endpoint/CubeVisitService.java
index c66af66..3b26b71 100644
--- 
a/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/cube/v2/coprocessor/endpoint/CubeVisitService.java
+++ 
b/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/cube/v2/coprocessor/endpoint/CubeVisitService.java
@@ -394,10 +394,6 @@ public class CubeVisitService extends 
CubeVisitProtos.CubeVisitService implement
 logger.error(ioe.toString(), ioe);
 IOException wrapped = new IOException("Error in coprocessor " + 
debugGitTag, ioe);
 ResponseConverter.setControllerException(controller, wrapped);
-} catch (OutOfMemoryError oom) {
-logger.error(oom.toString(), oom);
-IOException wrapped = new IOException("OOM in coprocessor " + 
debugGitTag, oom);
-ResponseConverter.setControllerException(controller, wrapped);
 } finally {
 for (RegionScanner innerScanner : regionScanners) {
 IOUtils.closeQuietly(innerScanner);