Skip to content

[BUG][Plugin] Fix #6167#6177

Merged
ruanwenjun merged 1 commit intoapache:devfrom
CalvinKirs:process
Sep 11, 2021
Merged

[BUG][Plugin] Fix #6167#6177
ruanwenjun merged 1 commit intoapache:devfrom
CalvinKirs:process

Conversation

@CalvinKirs
Copy link
Copy Markdown
Member

Fix #6167

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #6177 (7f8514a) into dev (ea2a9db) will decrease coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##                dev    #6177      +/-   ##
============================================
- Coverage     39.11%   39.06%   -0.05%     
+ Complexity     3254     3250       -4     
============================================
  Files           637      637              
  Lines         25282    25281       -1     
  Branches       2744     2744              
============================================
- Hits           9888     9877      -11     
- Misses        14511    14522      +11     
+ Partials        883      882       -1     
Impacted Files Coverage Δ
...apache/dolphinscheduler/spi/task/AbstractTask.java 0.00% <ø> (ø)
...ache/dolphinscheduler/server/log/LoggerServer.java 78.26% <0.00%> (-8.70%) ⬇️
...r/plugin/registry/zookeeper/ZookeeperRegistry.java 39.00% <0.00%> (-5.68%) ⬇️
...er/master/dispatch/host/assign/RandomSelector.java 77.77% <0.00%> (-5.56%) ⬇️

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 0715be3...7f8514a. 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 3 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

Copy link
Copy Markdown
Member

@ruanwenjun ruanwenjun left a comment

Choose a reason for hiding this comment

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

LGTM

@ruanwenjun ruanwenjun merged commit 58b694a into apache:dev Sep 11, 2021
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.

3 participants