Skip to content
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][scheduler] Support task group selection from DolphinScheduler #3418

Closed
3 tasks done
Yangxiaotian opened this issue Apr 23, 2024 · 2 comments · Fixed by #3419
Closed
3 tasks done

[Feature][scheduler] Support task group selection from DolphinScheduler #3418

Yangxiaotian opened this issue Apr 23, 2024 · 2 comments · Fixed by #3419
Assignees
Labels
New Feature New feature
Milestone

Comments

@Yangxiaotian
Copy link
Contributor

Yangxiaotian commented Apr 23, 2024

Search before asking

  • I had searched in the issues and found no similar feature requirement.

Description

  1. The form of sending a task to DolphinScheduler will be added a new selection field -- task groups of DS, and after sending the taskGroupId value will write into DS's database. The task groups is showed relatived to current project code.

image
3. Fix a bug: when the form window is reopened after sending to DS, some field values are not correctly shown even though the request's callback contains the correct value.
fcae52dda3ea8c6b5297a78553a9212

Use case

No response

Related issues

No response

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@Yangxiaotian Yangxiaotian added New Feature New feature Waiting for reply Waiting for reply labels Apr 23, 2024
@Zzm0809 Zzm0809 removed the Waiting for reply Waiting for reply label Apr 23, 2024
@Zzm0809 Zzm0809 moved this to Doing in Dinky Roadmap Apr 23, 2024
@Zzm0809 Zzm0809 added this to the 1.0.2 milestone Apr 23, 2024
@Zzm0809
Copy link
Contributor

Zzm0809 commented Apr 24, 2024

I saw that there is also an intra-group priority configuration for task groups in ds. Would you consider adding it together?
If you add a new one, you need to pay attention to selecting the task group before entering the priority.

image

@Yangxiaotian
Copy link
Contributor Author

OK, we will add the priority field.

@Zzm0809 Zzm0809 closed this as completed Apr 25, 2024
@Zzm0809 Zzm0809 linked a pull request Apr 25, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New Feature New feature
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants