Skip to content

[Fix][UI Next][V1.0.0-Alpha]: Fix the dag map display uncorrect afte…#8895

Merged
songjianet merged 1 commit into
apache:devfrom
Amy0104:fix-8877
Mar 15, 2022
Merged

[Fix][UI Next][V1.0.0-Alpha]: Fix the dag map display uncorrect afte…#8895
songjianet merged 1 commit into
apache:devfrom
Amy0104:fix-8877

Conversation

@Amy0104
Copy link
Copy Markdown
Member

@Amy0104 Amy0104 commented Mar 15, 2022

Purpose of the pull request

This pull request fixs the dag map display uncorrect after changing the pre tasks.

Releated issues

close #8877

@songjianet songjianet added bug Something isn't working UI ui and front end related labels Mar 15, 2022
@mergeable mergeable Bot removed bug Something isn't working UI ui and front end related labels Mar 15, 2022
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #8895 (c4b0d30) into dev (db64dca) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

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

Impacted file tree graph

@@             Coverage Diff              @@
##                dev    #8895      +/-   ##
============================================
- Coverage     45.30%   45.27%   -0.03%     
+ Complexity     4082     4081       -1     
============================================
  Files           702      702              
  Lines         27177    27186       +9     
  Branches       2907     2908       +1     
============================================
- Hits          12312    12309       -3     
- Misses        13704    13715      +11     
- Partials       1161     1162       +1     
Impacted Files Coverage Δ
...nscheduler/api/controller/ResourcesController.java 57.89% <0.00%> (-1.04%) ⬇️
...heduler/api/service/impl/ResourcesServiceImpl.java 53.66% <0.00%> (-0.68%) ⬇️
...e/dolphinscheduler/remote/NettyRemotingClient.java 50.70% <0.00%> (-2.82%) ⬇️
...dolphinscheduler/remote/future/ResponseFuture.java 83.60% <0.00%> (+1.63%) ⬆️

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 db64dca...68ddaff. Read the comment docs.

Copy link
Copy Markdown
Contributor

@devosend devosend left a comment

Choose a reason for hiding this comment

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

LGTM

@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

@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

@songjianet songjianet merged commit 8bcbe2a into apache:dev Mar 15, 2022
@Amy0104 Amy0104 deleted the fix-8877 branch April 12, 2022 09:19
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] [UI Next][V1.0.0-Alpha] The dag map displays error when I changed the pre tasks.

4 participants