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

support container znodes #334

Open
rgs1 opened this issue Jun 6, 2015 · 5 comments · May be fixed by #613
Open

support container znodes #334

rgs1 opened this issue Jun 6, 2015 · 5 comments · May be fixed by #613

Comments

@rgs1
Copy link
Contributor

rgs1 commented Jun 6, 2015

This is now merged on trunk and on the 3.5 branch of ZooKeeper: https://issues.apache.org/jira/browse/ZOOKEEPER-2163 (see apache/zookeeper@48f0269).

It might be backported for the upcoming 3.4.7 release too, so it would be nice to support it.

@teeeg
Copy link

teeeg commented Jun 18, 2019

Would this be something that maintainers would want to add to Kazoo? Is it feasible at this point in time? I would have an interest in doing the work to support containers if there's interest from the community.

@rgs1
Copy link
Contributor Author

rgs1 commented Jun 18, 2019

@teeeg i am happy to help with reviews.

@teeeg
Copy link

teeeg commented Jun 18, 2019

@rgs1 do you have an idea of the feasibility of supporting containers? how much work do you suppose it is?

@StephenSorriaux
Copy link
Member

@teeeg Yes, that would be a nice feature.

@rgs1
Copy link
Contributor Author

rgs1 commented Jun 18, 2019

@teeeg should be pretty straight fwd, given that container requests are basically create requests with a different opcode:

https://github.com/envoyproxy/envoy/blob/master/source/extensions/filters/network/zookeeper_proxy/filter.cc#L134

ceache added a commit to ceache/kazoo that referenced this issue Jun 16, 2020
@ceache ceache linked a pull request Jun 16, 2020 that will close this issue
ceache added a commit to ceache/kazoo that referenced this issue Jun 16, 2020
ceache added a commit to ceache/kazoo that referenced this issue Jun 16, 2020
ceache added a commit to ceache/kazoo that referenced this issue Jun 16, 2020
ceache added a commit to ceache/kazoo that referenced this issue Jun 16, 2020
ceache added a commit to ceache/kazoo that referenced this issue Jun 2, 2022
ceache added a commit to ceache/kazoo that referenced this issue Jun 2, 2022
ceache added a commit to ceache/kazoo that referenced this issue Jun 2, 2022
ceache added a commit to ceache/kazoo that referenced this issue Jun 2, 2022
ceache added a commit to ceache/kazoo that referenced this issue Jun 2, 2022
ceache added a commit to ceache/kazoo that referenced this issue Jun 2, 2022
ceache added a commit to ceache/kazoo that referenced this issue Jun 2, 2022
ceache added a commit to ceache/kazoo that referenced this issue Jun 2, 2022
ceache added a commit to ceache/kazoo that referenced this issue Feb 12, 2023
ceache added a commit to ceache/kazoo that referenced this issue Feb 12, 2023
ceache added a commit to ceache/kazoo that referenced this issue Feb 12, 2023
ceache added a commit to ceache/kazoo that referenced this issue Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants