Repository: kylin Updated Branches: refs/heads/KYLIN-2881-review ccbd8ac53 -> 61e4d3a76 (forced update)
minor, code review Project: http://git-wip-us.apache.org/repos/asf/kylin/repo Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/91c76dd2 Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/91c76dd2 Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/91c76dd2 Branch: refs/heads/KYLIN-2881-review Commit: 91c76dd2d28c6d198bafdb84db04190a7e796159 Parents: 3931157 Author: chenzhx <c...@apache.org> Authored: Mon Jan 8 14:01:47 2018 +0800 Committer: chenzhx <c...@apache.org> Committed: Mon Jan 8 14:02:11 2018 +0800 ---------------------------------------------------------------------- webapp/app/js/controllers/cubes.js | 20 +++++++++++++++++++- 1 file changed, 19 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/kylin/blob/91c76dd2/webapp/app/js/controllers/cubes.js ---------------------------------------------------------------------- diff --git a/webapp/app/js/controllers/cubes.js b/webapp/app/js/controllers/cubes.js index e6f4d71..c123f1c 100644 --- a/webapp/app/js/controllers/cubes.js +++ b/webapp/app/js/controllers/cubes.js @@ -597,7 +597,7 @@ var jobSubmitCtrl = function ($scope, $modalInstance, CubeService, MessageServic if (e.data && e.data.exception) { var message = e.data.exception; - if(message.indexOf("Empty cube segment found")!=-1 || message.indexOf("Merging segments must not have gaps between")!=-1){ + if(message.indexOf("Empty cube segment found")!=-1){ var _segment = message.substring(message.indexOf(":")+1,message.length-1); SweetAlert.swal({ title:'', @@ -610,11 +610,29 @@ var jobSubmitCtrl = function ($scope, $modalInstance, CubeService, MessageServic if (isConfirm) { $scope.jobBuildRequest.forceMergeEmptySegment = true; $scope.rebuild(); + delete $scope.jobBuildRequest.forceMergeEmptySegment; } }); return; } + if(message.indexOf("Merging segments must not have gaps between")!=-1){ + SweetAlert.swal({ + title:'', + type:'info', + text: 'There ares gaps between segments, do you want to merge segments forcely ?', + showCancelButton: true, + confirmButtonColor: '#DD6B55', + closeOnConfirm: true + }, function (isConfirm) { + if (isConfirm) { + $scope.jobBuildRequest.forceMergeEmptySegment = true; + $scope.rebuild(); + delete $scope.jobBuildRequest.forceMergeEmptySegment; + } + }); + return; + } var msg = !!(message) ? message : 'Failed to take action.'; SweetAlert.swal('Oops...', msg, 'error'); } else {