The Wayback Machine - https://web.archive.org/web/20210111203737/https://github.com/twitter/util/issues/207
Skip to content

/ util

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

Jump to bottom

Is there a ZooKeeper recursive path create function utils? #207

Open
ghost opened this issue Jan 12, 2018 · 6 comments
Open

Is there a ZooKeeper recursive path create function utils? #207

ghost opened this issue Jan 12, 2018 · 6 comments
Labels
good first issue help wanted

Comments

@ghost
Copy link

@ghost ghost commented Jan 12, 2018 •
edited by ghost

Hello,

I am working with the ZkAsyncSemaphore under zk-utils and I found that I need to pre-create the znodes in ZooKeeper or an exception will be thrown. Is there a defined utility function to do this? I found one in the commons library but it uses an older client.

EDIT:

Upon investigation, it seems that there is a function safeCreate within the semaphore that is meant to create the path for you. However, it doesn't seem to be working. I get a NoNode error from ZooKeeper.
@ryanoneill
Copy link
Contributor

@ryanoneill ryanoneill commented Jan 16, 2018

Hi @rquinlivan,

It appears that we no longer use ZkAsyncSemaphore in our code internally. It's quite possible that this code has grown stale as we've updated versions of Zookeeper.

And, I just tried manually using ZkAsyncSemaphore via the repl and running the tests with an instance of Zookeeper via the "com.twitter.zk.TEST_CONNECT" system property, and both failed in the exact method that you described.
@ryanoneill
Copy link
Contributor

@ryanoneill ryanoneill commented Jan 17, 2018

@rquinlivan, is this something you would be interested in fixing via a pull request?
@ghost
Copy link
Author

@ghost ghost commented Apr 9, 2018

Sure I will put a PR together
@luciferous
Copy link
Contributor

@luciferous luciferous commented Jul 16, 2019

Hi @rquinlivan are you still working on this?
@anukul
Copy link

@anukul anukul commented Aug 5, 2019

Hi! I'd like to take this up, if nobody else is working on it at the moment.
@kevinoliver
Copy link
Member

@kevinoliver kevinoliver commented Aug 5, 2019

@anukul given how long its been, I think it's safe for you to take a crack at it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue help wanted
6 participants
@luciferous @mosesn @kevinoliver @ryanoneill @yufangong @anukul
You can’t perform that action at this time.