Browse Source

[Fix-7070][UI] Remedy the issue with no saving the timeout's strategy into the database. (#7107)

3.0.0/version-upgrade
Hua Jiang 3 years ago committed by GitHub
parent
commit
d7eb7453e5
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 2
      dolphinscheduler-dao/src/main/java/org/apache/dolphinscheduler/dao/entity/TaskDefinition.java
  2. 24
      dolphinscheduler-ui/src/js/conf/home/pages/dag/_source/formModel/_source/timeoutAlarm.vue

2
dolphinscheduler-dao/src/main/java/org/apache/dolphinscheduler/dao/entity/TaskDefinition.java

@ -33,6 +33,7 @@ import java.util.Map;
import java.util.Objects; import java.util.Objects;
import java.util.stream.Collectors; import java.util.stream.Collectors;
import com.baomidou.mybatisplus.annotation.FieldStrategy;
import com.baomidou.mybatisplus.annotation.IdType; import com.baomidou.mybatisplus.annotation.IdType;
import com.baomidou.mybatisplus.annotation.TableField; import com.baomidou.mybatisplus.annotation.TableField;
import com.baomidou.mybatisplus.annotation.TableId; import com.baomidou.mybatisplus.annotation.TableId;
@ -158,6 +159,7 @@ public class TaskDefinition {
/** /**
* timeout notify strategy * timeout notify strategy
*/ */
@TableField(updateStrategy = FieldStrategy.IGNORED)
private TaskTimeoutStrategy timeoutNotifyStrategy; private TaskTimeoutStrategy timeoutNotifyStrategy;
/** /**

24
dolphinscheduler-ui/src/js/conf/home/pages/dag/_source/formModel/_source/timeoutAlarm.vue

@ -60,7 +60,11 @@
<script> <script>
import _ from 'lodash' import _ from 'lodash'
import disabledState from '@/module/mixin/disabledState' import disabledState from '@/module/mixin/disabledState'
const StrategyMap = {
WARN: 'WARN',
FAILED: 'FAILED',
WARNFAILED: 'WARNFAILED'
}
export default { export default {
name: 'form-timeout-alarm', name: 'form-timeout-alarm',
data () { data () {
@ -100,10 +104,12 @@
strategy: (() => { strategy: (() => {
// Handling checkout sequence // Handling checkout sequence
let strategy = this.strategy let strategy = this.strategy
if (strategy.length === 2 && strategy[0] === 'FAILED') { if (strategy.length > 1) {
return [strategy[1], strategy[0]].join(',') return StrategyMap.WARNFAILED
} else if (strategy.length === 1) {
return strategy[0]
} else { } else {
return strategy.join(',') return ''
} }
})(), })(),
interval: parseInt(this.interval), interval: parseInt(this.interval),
@ -119,7 +125,15 @@
// Non-null objects represent backfill // Non-null objects represent backfill
if (!_.isEmpty(o) && o.timeout) { if (!_.isEmpty(o) && o.timeout) {
this.enable = o.timeout.enable || false this.enable = o.timeout.enable || false
this.strategy = _.split(o.timeout.strategy, ',') || ['WARN'] if (o.timeout.strategy) {
if (o.timeout.strategy === StrategyMap.WARNFAILED) {
this.strategy = [StrategyMap.WARN, StrategyMap.FAILED]
} else {
this.strategy = [o.timeout.strategy]
}
} else {
this.strategy = [StrategyMap.WARN]
}
this.interval = o.timeout.interval || null this.interval = o.timeout.interval || null
} }
}, },

Loading…
Cancel
Save