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

Remove zookeeper config as admin client is not using it #386

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chromy96
Copy link

@chromy96 chromy96 commented Nov 1, 2022

Removing dependencies to zookeeper as described in issue: #385

@chromy96
Copy link
Author

@mhratson Any input on this?

@mhratson
Copy link
Collaborator

@chromy96 thanks for the contribution and sorry for the delay.
Looks great, but i'll have to work on test coverage first before merging more contributions.

@mhratson
Copy link
Collaborator

@chromy96 sorry for delay, i'll have time for review this in the next couple weeks.

@doxsch doxsch mentioned this pull request Oct 26, 2023
@lisl2862
Copy link

lisl2862 commented Sep 5, 2024

I'm a bit late to discussion, but any chance this still could be reviewed @mhratson? 🙏

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.

3 participants