Re: ClientConfiguration doesn't check classpath

2016-06-29 Thread Christopher
Was thinking about this a bit more. One of the problems with classpath is that makes it a bit hard to pull different configs within the same JVM (multiple client connections). It can also come from unexpected jars, resulting in surprising behavior. And, it depends on the classloader used at runtime

Re: ClientConfiguration doesn't check classpath

2016-06-29 Thread Josh Elser
Being able to pull off of the classpath would certainly be nice when you want to interact with multiple instances from the same host (thus requiring different client confs). Christopher's #1 point would also be a similar solution to the same problem. I'm not sure if one or the other would be bette

Re: ClientConfiguration doesn't check classpath

2016-06-29 Thread Christopher
To get rid of the warning, don't use ClientConfiguration.loadDefault(). Unit tests should be self-contained, and not use the user's environment. Instead, use "new ClientConfiguration()". If you're still getting that warning, we need to fix it. That constructor shouldn't be reading any external conf

ClientConfiguration doesn't check classpath

2016-06-29 Thread Mike Drob
Devs, Is there a reason that ClientConfiguration looks in the user's home directory, and for environment variables ACCUMULO_CONF_DIR and ACCUMULO_HOME to find a configuration but can't be controlled via system properties or the classpath? I'm working on some unit tests and keep seeing 'WARN org.a

[GitHub] accumulo pull request #121: ACCUMULO-4353: Stabilize tablet assignment durin...

2016-06-29 Thread ShawnWalker
Github user ShawnWalker commented on a diff in the pull request: https://github.com/apache/accumulo/pull/121#discussion_r68977594 --- Diff: server/base/src/main/java/org/apache/accumulo/server/master/state/TabletLocationState.java --- @@ -93,6 +95,9 @@ public TServerInstance getSe

[GitHub] accumulo issue #121: ACCUMULO-4353: Stabilize tablet assignment during trans...

2016-06-29 Thread ShawnWalker
Github user ShawnWalker commented on the issue: https://github.com/apache/accumulo/pull/121 Some testing on a small development cluster before I wrote `SuspendedTabletsIT`, but nothing significantly different from what I has `SuspendedTabletsIT` perform: Start Accumulo, create table,

[GitHub] accumulo issue #121: ACCUMULO-4353: Stabilize tablet assignment during trans...

2016-06-29 Thread keith-turner
Github user keith-turner commented on the issue: https://github.com/apache/accumulo/pull/121 @ShawnWalker did you do any testing besides the IT? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not hav