Browse Source

[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 <mzjnumber1@163.com>
Co-authored-by: dailidong <dailidong66@gmail.com>
pull/3/MERGE
muzhongjiang 4 years ago committed by GitHub
parent
commit
3aa34bc723
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 5
      dolphinscheduler-ui/src/js/conf/home/pages/dag/_source/startingParam/index.vue
  2. 6
      dolphinscheduler-ui/src/js/conf/home/pages/projects/pages/definition/pages/list/_source/start.vue
  3. 6
      dolphinscheduler-ui/src/js/conf/home/pages/projects/pages/definition/pages/list/_source/timing.vue

5
dolphinscheduler-ui/src/js/conf/home/pages/dag/_source/startingParam/index.vue

@ -76,14 +76,9 @@
return '-' return '-'
}, },
_getNotifyGroupList () { _getNotifyGroupList () {
let notifyGroupListS = _.cloneDeep(this.store.state.dag.notifyGroupListS) || []
if (!notifyGroupListS.length) {
this.store.dispatch('dag/getNotifyGroupList').then(res => { this.store.dispatch('dag/getNotifyGroupList').then(res => {
this.notifyGroupList = res this.notifyGroupList = res
}) })
} else {
this.notifyGroupList = notifyGroupListS
}
}, },
_getWorkerGroupList () { _getWorkerGroupList () {
let stateWorkerGroupsList = this.store.state.security.workerGroupsListAll || [] let stateWorkerGroupsList = this.store.state.security.workerGroupsListAll || []

6
dolphinscheduler-ui/src/js/conf/home/pages/projects/pages/definition/pages/list/_source/start.vue

@ -243,16 +243,10 @@
}, },
_getNotifyGroupList () { _getNotifyGroupList () {
return new Promise((resolve, reject) => { return new Promise((resolve, reject) => {
let notifyGroupListS = _.cloneDeep(this.store.state.dag.notifyGroupListS) || []
if (!notifyGroupListS.length) {
this.store.dispatch('dag/getNotifyGroupList').then(res => { this.store.dispatch('dag/getNotifyGroupList').then(res => {
this.notifyGroupList = res this.notifyGroupList = res
resolve() resolve()
}) })
} else {
this.notifyGroupList = notifyGroupListS
resolve()
}
}) })
}, },
_getReceiver () { _getReceiver () {

6
dolphinscheduler-ui/src/js/conf/home/pages/projects/pages/definition/pages/list/_source/timing.vue

@ -280,8 +280,6 @@
_getNotifyGroupList () { _getNotifyGroupList () {
return new Promise((resolve, reject) => { return new Promise((resolve, reject) => {
let notifyGroupListS = _.cloneDeep(this.store.state.dag.notifyGroupListS) || []
if (!notifyGroupListS.length) {
this.store.dispatch('dag/getNotifyGroupList').then(res => { this.store.dispatch('dag/getNotifyGroupList').then(res => {
this.notifyGroupList = res this.notifyGroupList = res
if (this.notifyGroupList.length) { if (this.notifyGroupList.length) {
@ -290,10 +288,6 @@
reject(new Error(0)) reject(new Error(0))
} }
}) })
} else {
this.notifyGroupList = notifyGroupListS
resolve()
}
}) })
}, },
ok () { ok () {

Loading…
Cancel
Save