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

illegal argument exception with SignalClient #6

Open
ppjogani opened this issue Mar 19, 2015 · 2 comments
Open

illegal argument exception with SignalClient #6

ppjogani opened this issue Mar 19, 2015 · 2 comments

Comments

@ppjogani
Copy link

SignalClient sc = new SignalClient("xx.xx.xx.xx:2181", "spout");

gives a

Exception in thread "main" java.lang.IllegalArgumentException: Path must start with / character
    at org.apache.curator.utils.PathUtils.validatePath(PathUtils.java:54)
    at org.apache.curator.utils.PathUtils.validatePath(PathUtils.java:37)
    at org.apache.curator.utils.ZKPaths.fixForNamespace(ZKPaths.java:63)
    at org.apache.curator.framework.imps.NamespaceImpl.fixForNamespace(NamespaceImpl.java:82)
    at org.apache.curator.framework.imps.CuratorFrameworkImpl.fixForNamespace(CuratorFrameworkImpl.java:574)
    at org.apache.curator.framework.imps.ExistsBuilderImpl.forPath(ExistsBuilderImpl.java:139)
    at org.apache.curator.framework.imps.ExistsBuilderImpl.forPath(ExistsBuilderImpl.java:36)
    at backtype.storm.contrib.signals.client.SignalClient.send(SignalClient.java:36)
    at storm.signalclient.TestSignalClient.main(TestSignalClient.java:11)

this is contrary to the README usage example.
I did try adding /, did not help.

@L3O
Copy link

L3O commented Aug 8, 2017

Any update on this ?

@itsmkj
Copy link

itsmkj commented Feb 27, 2018

Just replace this.name with "/"+ this.name everywhere the path is used

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

No branches or pull requests

3 participants