Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dynamic Configuration Zookeeper Implementation #5040

Merged
merged 3 commits into from Jul 7, 2020
Merged

Dynamic Configuration Zookeeper Implementation #5040

merged 3 commits into from Jul 7, 2020

Conversation

songzhendong
Copy link
Contributor

ADd configure the dynamic configuration of zookeeper server

Add how to configure the dynamic configuration of zookeeper server
@wu-sheng wu-sheng requested a review from zhaoyuguang July 6, 2020 12:55
@codecov
Copy link

codecov bot commented Jul 6, 2020

Codecov Report

Merging #5040 into master will decrease coverage by 0.67%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #5040      +/-   ##
============================================
- Coverage     53.07%   52.40%   -0.68%     
+ Complexity     2971     2954      -17     
============================================
  Files          1410      773     -637     
  Lines         30611    19421   -11190     
  Branches       3405     1916    -1489     
============================================
- Hits          16246    10177    -6069     
+ Misses        13560     8396    -5164     
- Partials        805      848      +43     
Impacted Files Coverage Δ Complexity Δ
...p/server/core/analysis/metrics/MinLongMetrics.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (ø%)
...er/exporter/provider/grpc/GRPCExporterSetting.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (ø%)
...king/oap/server/configuration/api/ConfigTable.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (ø%)
...g/oap/server/cluster/plugin/etcd/EtcdEndpoint.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (ø%)
...skywalking/oap/server/receiver/envoy/als/Role.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (ø%)
.../server/receiver/envoy/als/DependencyResource.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (ø%)
...erver/cluster/plugin/consul/ConsulCoordinator.java 0.00% <0.00%> (-97.06%) 0.00% <0.00%> (ø%)
.../analysis/transform/Zipkin2SkyWalkingTransfer.java 0.00% <0.00%> (-90.91%) 0.00% <0.00%> (ø%)
...ywalking/oap/server/library/util/ConnectUtils.java 0.00% <0.00%> (-90.00%) 0.00% <0.00%> (ø%)
...p/server/core/analysis/metrics/MaxLongMetrics.java 0.00% <0.00%> (-87.50%) 0.00% <0.00%> (ø%)
... and 959 more

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 73ee310...0da4493. Read the comment docs.

Simplify the document.
@wu-sheng wu-sheng added this to the 8.1.0 milestone Jul 7, 2020
@wu-sheng wu-sheng added the documentation Provide documents related issue or pull request only. label Jul 7, 2020
@wu-sheng wu-sheng merged commit 6ba9c7f into apache:master Jul 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Provide documents related issue or pull request only.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants