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

Shell not working #427

Closed
milleruntime opened this issue Apr 13, 2018 · 3 comments
Closed

Shell not working #427

milleruntime opened this issue Apr 13, 2018 · 3 comments
Labels
blocker This issue blocks any release version labeled on it. bug This issue has been verified to be a bug.
Projects

Comments

@milleruntime
Copy link
Contributor

I built the latest master and ran using Uno. Trying to run the Shell prints this error on the command line (with no other errors in the logs):
[shell.Shell] ERROR: Path must not end with / character

Sounds like an error coming from ZooKeeper or an external library because I couldn't find that string in the Accumulo code base.

@milleruntime milleruntime added v2.0.0 blocker This issue blocks any release version labeled on it. bug This issue has been verified to be a bug. labels Apr 13, 2018
@jmark99
Copy link
Contributor

jmark99 commented Apr 14, 2018

@milleruntime, check the accumulo-client.properties file in uno/install/accumulo-2.0.0/conf. I ran into the same issue recently as well. I think It's related to the recent commit removing the use of deprecated ClientConfiguration (#412). I think if you set the username/password/instancename values in that config file things will work. They did for me. I believe Uno needs to be updated to fill in those values now that the recent update has been pushed.

@mikewalch
Copy link
Member

The shell works if you pass all information in via the command line or use accumulo-client.properties. However, maybe this is a bug in that something is missing and a better error message should be printed.

The reason this error is occurring for you is that the shell used to get Accumulo instance information (ie. instance name & zookeepers) from the accumulo-site.xml which is bad practice. This info now needs to be passed in on command line or set in accumulo-client.properties. If update Uno, it will set up this file. See commit below:

apache/fluo-uno@8fbe10a

@milleruntime
Copy link
Contributor Author

Good call. I didn't have latest Uno updates. Closing.

@ctubbsii ctubbsii added this to Done in 2.0.0 Jun 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocker This issue blocks any release version labeled on it. bug This issue has been verified to be a bug.
Projects
No open projects
2.0.0
  
Done
Development

No branches or pull requests

3 participants