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.

Sign up for GitHub

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

Jump to bottom

Make registry, configcenter, matadata-report share a zookeeper connection in dubbo-2.7 #3179

Closed
cvictory opened this issue Jan 10, 2019 · 1 comment
Closed

Make registry, configcenter, matadata-report share a zookeeper connection in dubbo-2.7 #3179

cvictory opened this issue Jan 10, 2019 · 1 comment
Assignees
@cvictory

Comments

@cvictory
Copy link
Contributor

In dubbo 2.7, we introduce config-center, metadata-report and we define SPI for them. There are zookeeper extension for config-center and medata-report.
So far, there are 3 connections that are used for registry, config-center, metadata-report .

We should use one connection that can be shared with registry, config-center, metadata-report .
@cvictory cvictory self-assigned this Jan 10, 2019
cvictory added a commit to cvictory/dubbo that referenced this issue Jan 10, 2019
@cvictory
fix apache#3179. Make registry, configcenter, matadata-report share a…
b62401c 
… zookeeper connection
cvictory added a commit to cvictory/dubbo that referenced this issue Jan 10, 2019
@cvictory
fix apache#3179. Make registry, configcenter, matadata-report share a…
385aab2 
… zookeeper connection
@cvictory
Copy link
Contributor Author

@chickenlj should you modify the configcenter to this way later.

cvictory added a commit to cvictory/dubbo that referenced this issue Jan 10, 2019
@cvictory
fix apache#3179. optimize registry, configcenter, matadata-report sha…
5aa31fa 
…re a zookeeper connection
cvictory added a commit to cvictory/dubbo that referenced this issue Jan 10, 2019
@cvictory
fix apache#3179. add licence
45832a7
@cvictory cvictory mentioned this issue Jan 12, 2019
Merged
cvictory added a commit to cvictory/dubbo that referenced this issue Jan 14, 2019
@cvictory
fix apache#3179 , apache#3205, apache#3218. modify review issue; when…
4a10a81 
… sharing zookeeper connection, it should judge zookeeperClient.isConnected()
@dubbo-bot dubbo-bot mentioned this issue Jan 18, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cvictory cvictory

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@cvictory