Skip to content

[Feature-8485][task-plugin] add EMR task plugin#8487

Closed
ronyang1985 wants to merge 7 commits into
apache:devfrom
ronyang1985:feature-emr-task-plugin
Closed

[Feature-8485][task-plugin] add EMR task plugin#8487
ronyang1985 wants to merge 7 commits into
apache:devfrom
ronyang1985:feature-emr-task-plugin

Conversation

@ronyang1985
Copy link
Copy Markdown
Contributor

Purpose of the pull request

pr for this issue #8485

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 #8487 (dfa6089) into dev (2a844dc) will decrease coverage by 0.02%.
The diff coverage is 51.81%.

❗ Current head dfa6089 differs from pull request most recent head a6d8394. Consider uploading reports for the commit a6d8394 to get more accurate results

Impacted file tree graph

@@             Coverage Diff              @@
##                dev    #8487      +/-   ##
============================================
- Coverage     45.41%   45.39%   -0.03%     
- Complexity     4032     4046      +14     
============================================
  Files           686      691       +5     
  Lines         26725    26834     +109     
  Branches       2870     2880      +10     
============================================
+ Hits          12138    12182      +44     
- Misses        13441    13499      +58     
- Partials       1146     1153       +7     
Impacted Files Coverage Δ
...olphinscheduler/common/task/emr/EmrParameters.java 0.00% <0.00%> (ø)
...hinscheduler/common/utils/TaskParametersUtils.java 52.17% <0.00%> (-2.38%) ⬇️
...pache/dolphinscheduler/spi/task/TaskConstants.java 0.00% <ø> (ø)
...lphinscheduler/plugin/task/emr/EmrTaskChannel.java 0.00% <0.00%> (ø)
...heduler/plugin/task/emr/EmrTaskChannelFactory.java 0.00% <0.00%> (ø)
...ache/dolphinscheduler/plugin/task/emr/EmrTask.java 64.28% <64.28%> (ø)
...olphinscheduler/plugin/task/emr/EmrParameters.java 66.66% <66.66%> (ø)
...apache/dolphinscheduler/common/enums/TaskType.java 96.00% <100.00%> (+0.16%) ⬆️
...r/plugin/registry/zookeeper/ZookeeperRegistry.java 48.21% <0.00%> (-7.15%) ⬇️
...e/dolphinscheduler/remote/NettyRemotingClient.java 50.70% <0.00%> (-2.82%) ⬇️

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 2a844dc...a6d8394. 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 9 Code Smells

51.3% 51.3% Coverage
8.8% 8.8% Duplication

@ronyang1985
Copy link
Copy Markdown
Contributor Author

some dependencies confict need to check , close

@ronyang1985 ronyang1985 deleted the feature-emr-task-plugin branch February 22, 2022 14:21
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.

2 participants