This is an automated email from the ASF dual-hosted git repository. liyang pushed a commit to branch kylin5 in repository https://gitbox.apache.org/repos/asf/kylin.git
The following commit(s) were added to refs/heads/kylin5 by this push: new e734e6e31b KYLIN-5819,optimize the warning message on the dialog for editing computed column e734e6e31b is described below commit e734e6e31bd035c59c945ecfa99b136d2f6380b4 Author: huangchunyan <qingyanxiaon...@163.com> AuthorDate: Fri Apr 5 15:25:32 2024 +0800 KYLIN-5819,optimize the warning message on the dialog for editing computed column --- kystudio/src/components/common/GlobalDialog/dialog/detail_dialog.vue | 2 +- kystudio/src/components/studio/StudioModel/AddCCModal/locales.js | 2 +- .../components/studio/StudioModel/ModelList/ModelSaveConfig/index.vue | 4 ++-- .../studio/StudioModel/ModelList/ModelSaveConfig/locales.js | 3 ++- 4 files changed, 6 insertions(+), 5 deletions(-) diff --git a/kystudio/src/components/common/GlobalDialog/dialog/detail_dialog.vue b/kystudio/src/components/common/GlobalDialog/dialog/detail_dialog.vue index 1307c2f737..ec1f04335c 100644 --- a/kystudio/src/components/common/GlobalDialog/dialog/detail_dialog.vue +++ b/kystudio/src/components/common/GlobalDialog/dialog/detail_dialog.vue @@ -91,7 +91,7 @@ <template v-else> <el-button v-if="needResolveCancel" :type="isSubSubmit? 'primary': ''" :text="isSubSubmit" @click="handleCloseAndResove">{{cancelT}}</el-button> <el-button :type="isSubSubmit? 'primary': ''" :text="isSubSubmit" v-else :class="[needResolveCancel && 'ksd-ml-12']" @click="handleClose">{{cancelT}}</el-button> - <el-button v-if="isSubSubmit" :loading="loading&&!isBulidLoading" :disabled="loading&&isBulidLoading" class="ksd-ml-12" @click="handleSubmit(false)">{{submitSubText}}</el-button> + <el-button v-if="isSubSubmit" :type="isHideSubmit? 'primary': ''" :loading="loading&&!isBulidLoading" :disabled="loading&&isBulidLoading" class="ksd-ml-12" @click="handleSubmit(false)">{{submitSubText}}</el-button> <el-button type="primary" v-if="!isHideSubmit" class="ksd-ml-12" :loading="loading&&isBulidLoading" :disabled="loading&&!isBulidLoading" @click="handleSubmit(true)">{{submitT}}</el-button> </template> </div> diff --git a/kystudio/src/components/studio/StudioModel/AddCCModal/locales.js b/kystudio/src/components/studio/StudioModel/AddCCModal/locales.js index e437f7077b..87988ac338 100644 --- a/kystudio/src/components/studio/StudioModel/AddCCModal/locales.js +++ b/kystudio/src/components/studio/StudioModel/AddCCModal/locales.js @@ -1,5 +1,5 @@ export default { en: { - editCCTip: 'Please be aware that the modified expression would be effective until all the related indexes have been built.' + editCCTip: 'Modify with caution, it may cause query performance degradation! If the edited computed column is used as a dimension or metric, the system will delete the relevant index and generate new index. The query performance will not be restored until the build index job is done.' } } diff --git a/kystudio/src/components/studio/StudioModel/ModelList/ModelSaveConfig/index.vue b/kystudio/src/components/studio/StudioModel/ModelList/ModelSaveConfig/index.vue index 13839e9a1c..d8ded0cc7b 100644 --- a/kystudio/src/components/studio/StudioModel/ModelList/ModelSaveConfig/index.vue +++ b/kystudio/src/components/studio/StudioModel/ModelList/ModelSaveConfig/index.vue @@ -785,13 +785,13 @@ export default class ModelPartitionModal extends Vue { try { const res = await this.callGlobalDetailDialog({ msg: this.$t('editCCBuildTip'), - title: this.$t('kylinLang.common.tip'), + title: this.$t('editCCBuildTipTitle'), dialogType: 'warning', showDetailBtn: false, isSubSubmit: true, + isHideSubmit: true, wid: '600px', submitSubText: this.$t('kylinLang.common.save'), - submitText: this.$t('saveAndLoad'), needConcelReject: true }) this.handleClose(true, res.isOnlySave) diff --git a/kystudio/src/components/studio/StudioModel/ModelList/ModelSaveConfig/locales.js b/kystudio/src/components/studio/StudioModel/ModelList/ModelSaveConfig/locales.js index 441fb30e54..1bd439fbde 100644 --- a/kystudio/src/components/studio/StudioModel/ModelList/ModelSaveConfig/locales.js +++ b/kystudio/src/components/studio/StudioModel/ModelList/ModelSaveConfig/locales.js @@ -38,7 +38,8 @@ export default { incrementalTips: 'It will load data incrementally based on the selected partition column, which is more resource-efficient.', fullLoadTips: 'The system will load all data', changeBuildTypeTips: 'With partition setting changed, all segments and data would be deleted. The model couldn\'t serve queries. Meanwhile, the related ongoing jobs for building index would be discarded.', - editCCBuildTip: 'The modified expression of computed column would be effective until all the related indexes have been built. Do you want to save and build index now? ', + editCCBuildTipTitle: 'Will delete and generate new indexes, continue to save model?', + editCCBuildTip: 'Query performance will degrade until the new index are built! Modification of a computed column as a dimension or metric in the current model will cause the system to delete the relevant index and generate new index. The query performance will not be restored until the build index job is done.', saveAndBuild: 'Save and Build', purgeSegmentDataTips: 'Model definition has changed. Once saving the model, all data in the segments will be deleted. As a result, this model CAN\'T be used to serve queries. We strongly recommend to reload all data (in total {storageSize}).\r\n Do you want to continue?', onlyAddLeftJoinTip: 'The model definition relationship has changed and the current change only affects incremental data. To overwrite the stock data, go to the Segment page to clear the old data and rebuild.',