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 2022/12/05 10:21:06 UTC

[kylin] 14/22: check project admin permission

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 2902a64d43600bfae46e81890df751f37748c8da
Author: Zhixiong Chen <ch...@apache.org>
AuthorDate: Sat Oct 15 22:40:20 2022 +0800

    check project admin permission
---
 .../io/kyligence/kap/secondstorage/management/SecondStorageService.java  | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/second-storage/core-ui/src/main/java/io/kyligence/kap/secondstorage/management/SecondStorageService.java b/src/second-storage/core-ui/src/main/java/io/kyligence/kap/secondstorage/management/SecondStorageService.java
index b62b4e2376..c10c6c36ea 100644
--- a/src/second-storage/core-ui/src/main/java/io/kyligence/kap/secondstorage/management/SecondStorageService.java
+++ b/src/second-storage/core-ui/src/main/java/io/kyligence/kap/secondstorage/management/SecondStorageService.java
@@ -1174,6 +1174,7 @@ public class SecondStorageService extends BasicService implements SecondStorageU
     }
 
     public void modifyColumn(String project, String model, String column, String datatype) {
+        isProjectAdmin(project);
         logger.info("Start to modify second storage low cardinality on model {}.", model);
         if (!SecondStorageUtil.isProjectEnable(project) || !SecondStorageUtil.isModelEnable(project, model)) {
             throw new KylinException(INVALID_PARAMETER, String.format("The model does not have tiered storage enabled on project %s.", project));