Skip to content

[Fix][UI Next][V1.0.0-Alpha] Fix the deploy mode with an extra client option in FLINK.#9131

Merged
songjianet merged 1 commit into
apache:devfrom
Amy0104:fix-9130
Mar 23, 2022
Merged

[Fix][UI Next][V1.0.0-Alpha] Fix the deploy mode with an extra client option in FLINK.#9131
songjianet merged 1 commit into
apache:devfrom
Amy0104:fix-9130

Conversation

@Amy0104
Copy link
Copy Markdown
Member

@Amy0104 Amy0104 commented Mar 23, 2022

Purpose of the pull request

This pull request fixs the deploy mode with an extra client option in FLINK.

Releated issues

close #9130

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

Codecov Report

Merging #9131 (5e8f750) into dev (aae328e) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #9131      +/-   ##
============================================
- Coverage     40.28%   40.28%   -0.01%     
+ Complexity     4366     4365       -1     
============================================
  Files           808      808              
  Lines         32547    32547              
  Branches       3646     3646              
============================================
- Hits          13113    13112       -1     
  Misses        18208    18208              
- Partials       1226     1227       +1     
Impacted Files Coverage Δ
...dolphinscheduler/remote/future/ResponseFuture.java 81.96% <0.00%> (-1.64%) ⬇️

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 d2138c7...5e8f750. Read the comment docs.

@sonarqubecloud
Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

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

No Coverage information No Coverage information
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 08ea1aa into apache:dev Mar 23, 2022
@Amy0104 Amy0104 deleted the fix-9130 branch March 24, 2022 04:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working UI ui and front end related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug-FE][UI Next][V1.0.0-Alpha] The deploy mode shouldn't has the client option in FLINK.

3 participants