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

Project-10: Replace global ZooKeeper and cache access from PulsarService and replace with Metadata-store API #9643

Closed
rdhabalia opened this issue Feb 19, 2021 · 0 comments · Fixed by #9648
Closed

Project-10: Replace global ZooKeeper and cache access from PulsarService and replace with Metadata-store API #9643

rdhabalia opened this issue Feb 19, 2021 · 0 comments · Fixed by #9648
Assignees
@rdhabalia
Labels
type/enhancement The enhancements for the existing features or docs. e.g. reduce memory usage of the delayed messages
Milestone
2.8.0

Comments

@rdhabalia
Copy link
Contributor

rdhabalia commented Feb 19, 2021 •
edited
Loading

Goal

PulsarService initializes Global-ZooKeeper and that is used by multiple components across pulsar-services. Remove global-zk access from dependent component and eventually remove global-zk initialization from the PulsarService.

Reference

Component: Pulsar-Service
Class-reference: "- PulsarServie::globalZkCache, ZooKeeperCache
Affected Modules: "- BrokerService, NamespaceServicem PersistentTopic, WorkerService

Notes

  1. PulsarServie::globalZkCache initializes global zk and that is being used by multiple places eg: BrokerService, NamespaceServicem PersistentTopic, WorkerService
  2. Replace globalZk access with Metadata-store resources in those places
  3. Remove GlogalZK reference from the PulsarService
@rdhabalia rdhabalia added the type/enhancement The enhancements for the existing features or docs. e.g. reduce memory usage of the delayed messages label Feb 19, 2021
@rdhabalia rdhabalia added this to the 2.8.0 milestone Feb 19, 2021
@rdhabalia rdhabalia self-assigned this Feb 19, 2021
@sijie sijie mentioned this issue Feb 19, 2021
Closed
@rdhabalia rdhabalia mentioned this issue Feb 20, 2021
Merged
@rdhabalia rdhabalia linked a pull request Feb 20, 2021 that will close this issue
[pulsar-broker] Remove global-zk reference from Pulsar-service #9648
Merged
This was referenced Oct 16, 2021
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rdhabalia rdhabalia

Labels
type/enhancement The enhancements for the existing features or docs. e.g. reduce memory usage of the delayed messages
Projects
None yet
Milestone
2.8.0
Development

Successfully merging a pull request may close this issue.

[pulsar-broker] Remove global-zk reference from Pulsar-service rdhabalia/pulsar
1 participant
@rdhabalia