You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by xx...@apache.org on 2023/05/06 06:59:27 UTC

[kylin] 26/38: KYLIN-5533 [FOLLOW UP] fix BeanUtils copyProperties NPE

This is an automated email from the ASF dual-hosted git repository.

xxyu pushed a commit to branch kylin5
in repository https://gitbox.apache.org/repos/asf/kylin.git

commit c8a464d2cadc72f56fe51d98f2914eecdfe00e22
Author: lixiang <44...@qq.com>
AuthorDate: Tue Feb 28 17:56:52 2023 +0800

    KYLIN-5533 [FOLLOW UP] fix BeanUtils copyProperties NPE
---
 .../src/main/java/org/apache/kylin/rest/service/ModelService.java       | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/modeling-service/src/main/java/org/apache/kylin/rest/service/ModelService.java b/src/modeling-service/src/main/java/org/apache/kylin/rest/service/ModelService.java
index c3645f4af8..21d52c7956 100644
--- a/src/modeling-service/src/main/java/org/apache/kylin/rest/service/ModelService.java
+++ b/src/modeling-service/src/main/java/org/apache/kylin/rest/service/ModelService.java
@@ -695,7 +695,7 @@ public class ModelService extends AbstractModelService implements TableModelSupp
             try {
                 NDataModelResponse nDataModelResponse = convertToDataModelResponse(dataModel, projectName, dfManager,
                         status, queryElem.isOnlyNormalDim());
-                if (lite) {
+                if (lite && nDataModelResponse != null) {
                     return new NDataModelLiteResponse(nDataModelResponse, dataModel);
                 } else {
                     return nDataModelResponse;