This is an automated email from the ASF dual-hosted git repository.
shaofengshi pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kylin.git
The following commit(s) were added to refs/heads/master by this push:
new 64dfa7b KYLIN-3507 Avoid NPE when project is not found
64dfa7b is described below
commit 64dfa7b9f7ef95a3edd8548d46a21ea9e71facc7
Author: hujixu <[email protected]>
AuthorDate: Wed Aug 22 14:59:21 2018 +0800
KYLIN-3507 Avoid NPE when project is not found
---
.../src/main/java/org/apache/kylin/rest/service/QueryService.java | 6 ++++++
1 file changed, 6 insertions(+)
diff --git
a/server-base/src/main/java/org/apache/kylin/rest/service/QueryService.java
b/server-base/src/main/java/org/apache/kylin/rest/service/QueryService.java
index f195e74..16358e4 100644
--- a/server-base/src/main/java/org/apache/kylin/rest/service/QueryService.java
+++ b/server-base/src/main/java/org/apache/kylin/rest/service/QueryService.java
@@ -86,6 +86,7 @@ import org.apache.kylin.metadata.model.JoinTableDesc;
import org.apache.kylin.metadata.model.ModelDimensionDesc;
import org.apache.kylin.metadata.model.TableRef;
import org.apache.kylin.metadata.project.ProjectInstance;
+import org.apache.kylin.metadata.project.ProjectManager;
import org.apache.kylin.metadata.querymeta.ColumnMeta;
import org.apache.kylin.metadata.querymeta.ColumnMetaWithType;
import org.apache.kylin.metadata.querymeta.SelectedColumnMeta;
@@ -359,6 +360,11 @@ public class QueryService extends BasicService {
if (StringUtils.isBlank(sqlRequest.getProject())) {
throw new BadRequestException(msg.getEMPTY_PROJECT_NAME());
}
+ // project not found
+ ProjectManager mgr =
ProjectManager.getInstance(KylinConfig.getInstanceFromEnv());
+ if (mgr.getProject(sqlRequest.getProject()) == null) {
+ throw new BadRequestException(msg.getPROJECT_NOT_FOUND());
+ }
if (StringUtils.isBlank(sqlRequest.getSql())) {
throw new BadRequestException(msg.getNULL_EMPTY_SQL());
}