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

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

Comments

@ghost
Copy link

ghost commented Jan 12, 2018

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

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

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

@ghost
Copy link
Author

ghost commented Apr 9, 2018

Sure I will put a PR together

@luciferous
Copy link
Collaborator

Hi @rquinlivan are you still working on this?

@kevinoliver
Copy link
Contributor

@anukul given how long its been, I think it's safe for you to take a crack at it.

@BurakCakan
Copy link

Do you still need help here? If you are, I wanna look at it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

6 participants