Skip to content

[Fix-8980] [master] fixed repeat processing command#9220

Merged
EricJoy2048 merged 1 commit into
apache:devfrom
guoshupei:Bug-8980
Mar 27, 2022
Merged

[Fix-8980] [master] fixed repeat processing command#9220
EricJoy2048 merged 1 commit into
apache:devfrom
guoshupei:Bug-8980

Conversation

@guoshupei
Copy link
Copy Markdown
Contributor

This closes #8980

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.

(or)

This pull request is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #9220 (786f317) into dev (91319e3) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

❗ Current head 786f317 differs from pull request most recent head 6a117e7. Consider uploading reports for the commit 6a117e7 to get more accurate results

@@             Coverage Diff              @@
##                dev    #9220      +/-   ##
============================================
- Coverage     40.20%   40.18%   -0.02%     
+ Complexity     4375     4374       -1     
============================================
  Files           817      818       +1     
  Lines         32686    32697      +11     
  Branches       3642     3644       +2     
============================================
- Hits          13140    13139       -1     
- Misses        18316    18329      +13     
+ Partials       1230     1229       -1     
Impacted Files Coverage Δ
.../dolphinscheduler/common/enums/SlotCheckState.java 0.00% <0.00%> (ø)
...uler/server/master/registry/ServerNodeManager.java 1.28% <0.00%> (-0.01%) ⬇️
...r/server/master/runner/MasterSchedulerService.java 0.00% <0.00%> (ø)
...er/master/dispatch/host/assign/RandomSelector.java 77.77% <0.00%> (-5.56%) ⬇️
...r/plugin/task/sqoop/parameter/SqoopParameters.java 53.33% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 91319e3...6a117e7. Read the comment docs.

@sonarqubecloud
Copy link
Copy Markdown

SonarCloud Quality Gate failed.    Quality Gate failed

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

0.0% 0.0% Coverage
0.0% 0.0% Duplication

Copy link
Copy Markdown
Member

@EricJoy2048 EricJoy2048 left a comment

Choose a reason for hiding this comment

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

+1

@EricJoy2048
Copy link
Copy Markdown
Member

Thanks @guoshupei

@EricJoy2048 EricJoy2048 merged commit 258285e into apache:dev Mar 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] [Master] master can repeat processing command

3 participants