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.

Sign up for GitHub

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

Jump to bottom

[fix] Task group queue query all list error #10237

Merged
merged 2 commits into from
May 25, 2022
Merged

[fix] Task group queue query all list error #10237

merged 2 commits into from
May 25, 2022

Conversation

zhongjiajie
Copy link
Member

Add support query all task queue by not given specific groupId

close: #10102

@zhongjiajie zhongjiajie self-assigned this May 25, 2022
@zhongjiajie zhongjiajie added the bug Something isn't working label May 25, 2022
@mergeable mergeable bot removed the bug Something isn't working label May 25, 2022
@zhongjiajie zhongjiajie added this to the 3.0.0-beta-1 milestone May 25, 2022
@zhongjiajie zhongjiajie marked this pull request as ready for review May 25, 2022 03:40
@zhongjiajie zhongjiajie added bug Something isn't working backend labels May 25, 2022
@zhongjiajie zhongjiajie force-pushed the b-api-task-group-all branch 2 times, most recently from 23129bd to e1a6f9f Compare May 25, 2022 06:16
@zhongjiajie
[Fix] Task group queue query all list error
5d4fcb3 
Add support query all task queue by not given specific groupId

close: apache#10102
caishunfeng
caishunfeng approved these changes May 25, 2022
View reviewed changes
Copy link
Contributor

@caishunfeng caishunfeng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sonarcloud
Copy link

sonarcloud bot commented May 25, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@SbloodyS
Copy link
Member

I restart the failed CI.

@zhongjiajie zhongjiajie merged commit d8db2b5 into apache:dev May 25, 2022
@zhongjiajie zhongjiajie deleted the b-api-task-group-all branch May 25, 2022 08:33
zhongjiajie added a commit that referenced this pull request May 25, 2022
@zhongjiajie
[fix] Task group queue query all list error (#10237)
e663043 
Add support query all task queue by not given specific groupId

close: #10102
(cherry picked from commit d8db2b5)
ITBOX-ITBOY pushed a commit to ITBOX-ITBOY/dolphinscheduler that referenced this pull request Jul 8, 2022
@zhongjiajie
[fix] Task group queue query all list error (apache#10237)
04dca39 
Add support query all task queue by not given specific groupId

close: apache#10102
@woshiwuxiaofei woshiwuxiaofei mentioned this pull request Nov 1, 2022
Closed
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@caishunfeng caishunfeng caishunfeng approved these changes

@wangxj3 wangxj3 Awaiting requested review from wangxj3

@Tianqi-Dotes Tianqi-Dotes Awaiting requested review from Tianqi-Dotes

Assignees

@zhongjiajie zhongjiajie

Labels
backend bug Something isn't working
Projects
None yet
Milestone
3.0.0-beta-1
Development

Successfully merging this pull request may close these issues.

[Bug] [Api] Task Group Queue Query Error
3 participants
@zhongjiajie @SbloodyS @caishunfeng