From 3aa34bc72363eab368ade91b6e5be77b67897b02 Mon Sep 17 00:00:00 2001 From: muzhongjiang Date: Thu, 16 Jul 2020 14:56:53 +0800 Subject: [PATCH] [bugfix] "getNotifyGroupList" cache bug #3179 (#3200) * add state * fixed bug "jackson enum conversion : InvalidFormatException" * Word spelling modification Comment modification Word spelling modification,Comment modification,Log level modification * Update EmailManager.java * Update FlinkParameters.java * Update SqlTask.java * fixed "getNotifyGroupList cache" bug Co-authored-by: mzjnumber1@163.com Co-authored-by: dailidong --- .../js/conf/home/pages/dag/_source/startingParam/index.vue | 5 ----- .../projects/pages/definition/pages/list/_source/start.vue | 6 ------ .../projects/pages/definition/pages/list/_source/timing.vue | 6 ------ 3 files changed, 17 deletions(-) diff --git a/dolphinscheduler-ui/src/js/conf/home/pages/dag/_source/startingParam/index.vue b/dolphinscheduler-ui/src/js/conf/home/pages/dag/_source/startingParam/index.vue index bea5d92227..516f03bc2f 100644 --- a/dolphinscheduler-ui/src/js/conf/home/pages/dag/_source/startingParam/index.vue +++ b/dolphinscheduler-ui/src/js/conf/home/pages/dag/_source/startingParam/index.vue @@ -76,14 +76,9 @@ return '-' }, _getNotifyGroupList () { - let notifyGroupListS = _.cloneDeep(this.store.state.dag.notifyGroupListS) || [] - if (!notifyGroupListS.length) { this.store.dispatch('dag/getNotifyGroupList').then(res => { this.notifyGroupList = res }) - } else { - this.notifyGroupList = notifyGroupListS - } }, _getWorkerGroupList () { let stateWorkerGroupsList = this.store.state.security.workerGroupsListAll || [] diff --git a/dolphinscheduler-ui/src/js/conf/home/pages/projects/pages/definition/pages/list/_source/start.vue b/dolphinscheduler-ui/src/js/conf/home/pages/projects/pages/definition/pages/list/_source/start.vue index e47d8c757f..87c0f62cdb 100644 --- a/dolphinscheduler-ui/src/js/conf/home/pages/projects/pages/definition/pages/list/_source/start.vue +++ b/dolphinscheduler-ui/src/js/conf/home/pages/projects/pages/definition/pages/list/_source/start.vue @@ -243,16 +243,10 @@ }, _getNotifyGroupList () { return new Promise((resolve, reject) => { - let notifyGroupListS = _.cloneDeep(this.store.state.dag.notifyGroupListS) || [] - if (!notifyGroupListS.length) { this.store.dispatch('dag/getNotifyGroupList').then(res => { this.notifyGroupList = res resolve() }) - } else { - this.notifyGroupList = notifyGroupListS - resolve() - } }) }, _getReceiver () { diff --git a/dolphinscheduler-ui/src/js/conf/home/pages/projects/pages/definition/pages/list/_source/timing.vue b/dolphinscheduler-ui/src/js/conf/home/pages/projects/pages/definition/pages/list/_source/timing.vue index 82a82dec45..65b9176c18 100644 --- a/dolphinscheduler-ui/src/js/conf/home/pages/projects/pages/definition/pages/list/_source/timing.vue +++ b/dolphinscheduler-ui/src/js/conf/home/pages/projects/pages/definition/pages/list/_source/timing.vue @@ -280,8 +280,6 @@ _getNotifyGroupList () { return new Promise((resolve, reject) => { - let notifyGroupListS = _.cloneDeep(this.store.state.dag.notifyGroupListS) || [] - if (!notifyGroupListS.length) { this.store.dispatch('dag/getNotifyGroupList').then(res => { this.notifyGroupList = res if (this.notifyGroupList.length) { @@ -290,10 +288,6 @@ reject(new Error(0)) } }) - } else { - this.notifyGroupList = notifyGroupListS - resolve() - } }) }, ok () {