Skip to content

[DS-6655][fix] remove zkmonitor#6788

Closed
981433814 wants to merge 3 commits intoapache:devfrom
981433814:dev
Closed

[DS-6655][fix] remove zkmonitor#6788
981433814 wants to merge 3 commits intoapache:devfrom
981433814:dev

Conversation

@981433814
Copy link
Contributor

@981433814 981433814 commented Nov 11, 2021

remove zkmonitor,resolve the conflict

this pr close #6655

@codecov-commenter
Copy link

Codecov Report

Merging #6788 (f139564) into dev (088a5a4) will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##                dev    #6788      +/-   ##
============================================
+ Coverage     41.82%   41.85%   +0.02%     
  Complexity     3617     3617              
============================================
  Files           641      641              
  Lines         25897    25897              
  Branches       2795     2795              
============================================
+ Hits          10832    10838       +6     
+ Misses        14083    14078       -5     
+ Partials        982      981       -1     
Impacted Files Coverage Δ
...org/apache/dolphinscheduler/remote/utils/Host.java 37.77% <0.00%> (-2.23%) ⬇️
...e/dolphinscheduler/remote/NettyRemotingClient.java 52.11% <0.00%> (-1.41%) ⬇️
...dolphinscheduler/remote/future/ResponseFuture.java 83.05% <0.00%> (+1.69%) ⬆️
...r/plugin/registry/zookeeper/ZookeeperRegistry.java 54.54% <0.00%> (+7.27%) ⬆️

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 088a5a4...f139564. Read the comment docs.

@sonarqubecloud
Copy link

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
No Duplication information No Duplication information

@981433814 981433814 closed this Nov 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.

[Bug] [server-zkmonitor] zkmonitor can't show for normal

2 participants