-
Notifications
You must be signed in to change notification settings - Fork 4.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feature-14545][Master][UI]Cross workflow parameter passing #14552
[Feature-14545][Master][UI]Cross workflow parameter passing #14552
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #14552 +/- ##
============================================
- Coverage 38.48% 38.44% -0.04%
+ Complexity 4549 4547 -2
============================================
Files 1254 1254
Lines 43709 43743 +34
Branches 4822 4826 +4
============================================
- Hits 16822 16819 -3
- Misses 25013 25050 +37
Partials 1874 1874
... and 1 file with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
# Conflicts: # dolphinscheduler-master/src/main/java/org/apache/dolphinscheduler/server/master/runner/WorkflowExecuteRunnable.java
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please update the docs too.
...rc/main/java/org/apache/dolphinscheduler/server/master/runner/WorkflowExecuteThreadPool.java
Outdated
Show resolved
Hide resolved
...rc/main/java/org/apache/dolphinscheduler/server/master/runner/WorkflowExecuteThreadPool.java
Outdated
Show resolved
Hide resolved
Co-authored-by: xiangzihao <zihaoxiang@apache.org>
Co-authored-by: xiangzihao <zihaoxiang@apache.org>
...rc/main/java/org/apache/dolphinscheduler/server/master/runner/WorkflowExecuteThreadPool.java
Outdated
Show resolved
Hide resolved
Please retry analysis of this Pull-Request directly on SonarCloud. |
* feat: cross-workflow parameter passing * refactor: remove useless parameters * refactor: remove useless code * refactor: code format * docs: update docs * docs: docs format * more effective information Co-authored-by: xiangzihao <zihaoxiang@apache.org> * more effective information Co-authored-by: xiangzihao <zihaoxiang@apache.org> * fix: wrong writing of log code * fix: perfect error log --------- Co-authored-by: xiangzihao <zihaoxiang@apache.org> Co-authored-by: xiangzihao <460888207@qq.com> Co-authored-by: Rick Cheng <rickchengx@gmail.com> Co-authored-by: Jay Chung <zhongjiajie955@gmail.com> (cherry picked from commit af2986e)
…4552) * feat: cross-workflow parameter passing * refactor: remove useless parameters * refactor: remove useless code * refactor: code format * docs: update docs * docs: docs format * more effective information Co-authored-by: xiangzihao <zihaoxiang@apache.org> * more effective information Co-authored-by: xiangzihao <zihaoxiang@apache.org> * fix: wrong writing of log code * fix: perfect error log --------- Co-authored-by: xiangzihao <zihaoxiang@apache.org> Co-authored-by: xiangzihao <460888207@qq.com> Co-authored-by: Rick Cheng <rickchengx@gmail.com> Co-authored-by: Jay Chung <zhongjiajie955@gmail.com>
Purpose of the pull request
cross-workflow parameter passing
close #14545
Brief change log
add relevant code in module dolphinscheduler-ui
add code in
submitPostNode
method ofWorkflowExecuteRunnable.java
add method
crossWorkflowParameterPassing
inWorkflowExecuteThreadPool.java
Design and usage documentation can be seen here #14545
Verify this pull request
This change can be verified as follows:
Manually verified the change by testing locally.