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));