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 <c...@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));

Reply via email to