Skip to content

[Feature][UI Next]add subprocess task #8439

Merged
songjianet merged 4 commits into
apache:devfrom
labbomb:labbomb
Feb 19, 2022
Merged

[Feature][UI Next]add subprocess task #8439
songjianet merged 4 commits into
apache:devfrom
labbomb:labbomb

Conversation

@labbomb
Copy link
Copy Markdown
Member

@labbomb labbomb commented Feb 19, 2022

Purpose of the pull request

Please refer to the issue #7928.

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:

@songjianet songjianet added feature new feature UI ui and front end related labels Feb 19, 2022
Copy link
Copy Markdown
Member

@songjianet songjianet left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #8439 (74746e3) into dev (7da84e4) will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##                dev    #8439      +/-   ##
============================================
- Coverage     45.35%   45.34%   -0.02%     
+ Complexity     4028     4026       -2     
============================================
  Files           686      686              
  Lines         26702    26702              
  Branches       2867     2867              
============================================
- Hits          12112    12108       -4     
- Misses        13448    13450       +2     
- Partials       1142     1144       +2     
Impacted Files Coverage Δ
...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 7da84e4...74746e3. 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 2 Code Smells

0.0% 0.0% Coverage
68.3% 68.3% Duplication

@songjianet songjianet merged commit ced0116 into apache:dev Feb 19, 2022
@labbomb labbomb deleted the labbomb branch February 19, 2022 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature new feature UI ui and front end related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants