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

[Improvement][master-sql] Optimize the sql for queryTheHighestPriorityTasks #13671

Merged
merged 1 commit into from
Mar 5, 2023

Conversation

hiSandog
Copy link
Contributor

@hiSandog hiSandog commented Mar 5, 2023

The previous sql was not only semantically ambiguous, but also inefficient.
So I made some small improvements

@codecov-commenter
Copy link

codecov-commenter commented Mar 5, 2023

Codecov Report

Merging #13671 (34d6517) into dev (fd72088) will increase coverage by 0.01%.
The diff coverage is n/a.

❗ Current head 34d6517 differs from pull request most recent head 39e1c35. Consider uploading reports for the commit 39e1c35 to get more accurate results

@@             Coverage Diff              @@
##                dev   #13671      +/-   ##
============================================
+ Coverage     39.15%   39.17%   +0.01%     
  Complexity     4426     4426              
============================================
  Files          1129     1129              
  Lines         42121    42121              
  Branches       4776     4776              
============================================
+ Hits          16491    16499       +8     
+ Misses        23814    23805       -9     
- Partials       1816     1817       +1     
Impacted Files Coverage Δ
...r/plugin/task/sqoop/parameter/SqoopParameters.java 38.09% <0.00%> (-1.20%) ⬇️
...r/plugin/registry/zookeeper/ZookeeperRegistry.java 50.80% <0.00%> (+7.25%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@sonarcloud
Copy link

sonarcloud bot commented Mar 5, 2023

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
No Duplication information No Duplication information

@EricGao888 EricGao888 added improvement make more easy to user or prompt friendly 3.2.0 for 3.2.0 version labels Mar 5, 2023
@EricGao888 EricGao888 added this to the 3.2.0 milestone Mar 5, 2023
@EricGao888 EricGao888 merged commit c2d5774 into apache:dev Mar 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.2.0 for 3.2.0 version backend improvement make more easy to user or prompt friendly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants