[2/2] kylin git commit: KYLIN-2794 code review

2017-10-08 Thread liyang
KYLIN-2794 code review This closes #78 Project: http://git-wip-us.apache.org/repos/asf/kylin/repo Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/c04be637 Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/c04be637 Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/c04be637

[2/2] kylin git commit: KYLIN-2794 code review

2017-10-08 Thread liyang
KYLIN-2794 code review Project: http://git-wip-us.apache.org/repos/asf/kylin/repo Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/a82847ea Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/a82847ea Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/a82847ea Branch: refs/he