-
Notifications
You must be signed in to change notification settings - Fork 4.5k
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]Supports dinky tasks to accept Prepare Params passed within a task or from a previous task #15947
base: dev
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #15947 +/- ##
=========================================
Coverage 40.67% 40.67%
Complexity 5238 5238
=========================================
Files 1382 1382
Lines 46023 46023
Branches 4922 4922
=========================================
Hits 18719 18719
Misses 25379 25379
Partials 1925 1925 ☔ View full report in Codecov by Sentry. |
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 supplement the document. @gaoyan1998
@SbloodyS PTAL, Supplement the documentation done. |
Please run |
I have resolved it,please review it,thanks. @SbloodyS |
c5f64c7
to
bf68543
Compare
...eduler-task-dinky/src/main/java/org/apache/dolphinscheduler/plugin/task/dinky/DinkyTask.java
Outdated
Show resolved
Hide resolved
...eduler-task-dinky/src/main/java/org/apache/dolphinscheduler/plugin/task/dinky/DinkyTask.java
Outdated
Show resolved
Hide resolved
@ruanwenjun Thanks for your suggestion, I have finished the revision |
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.
LGTM
…ain/java/org/apache/dolphinscheduler/plugin/task/dinky/DinkyTask.java Co-authored-by: Wenjun Ruan <wenjun@apache.org>
…ain/java/org/apache/dolphinscheduler/plugin/task/dinky/DinkyTask.java Co-authored-by: Wenjun Ruan <wenjun@apache.org>
14cf7fb
to
2902943
Compare
…-params # Conflicts: # dolphinscheduler-task-plugin/dolphinscheduler-task-dinky/src/main/java/org/apache/dolphinscheduler/plugin/task/dinky/DinkyTask.java
hi @ruanwenjun PTAL ,please approval workflows |
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.
LGTM
Quality Gate passedIssues Measures |
Purpose of the pull request
Supports dinky tasks to accept variables passed within a task or from a previous task
Brief change log
Verify this pull request