apache / apache/dolphinscheduler

[Improvement][Master] Initialize WorkflowGraph/WorkflowExecutionGraph when `WorkflowStartLifecycleEvent` fired.

Open
#17,751 10 comments 0 reactions 1 assignee Assigned to @GaneshPatil7517 View on GitHub
improvement priority:middle
Dominant language
Java
Stars
14.5k
Forks
5.1k
Avg merge
1d 21h
Merged PRs (30d)
29

Description

### Search before asking

- [x] I had searched in the [issues](https://github.com/apache/dolphinscheduler/issues?q=is%3Aissue) and found no similar feature requirement.

### Description

It's better not initialize WorkflowGraph/WorkflowExecutionGraph when handle command, and initialize these when start event fired.
The advantage is that we can reduce transaction time when processing commands, and once initialize graph failed, we can trigger a workflow instance failure event.

### Are you willing to submit a PR?

- [x] Yes I am willing to submit a PR!

### Code of Conduct

- [x] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.