Re: Updates to Accumulo scripts for 2.0

2016-10-06 Thread Christopher
I would strongly prefer we keep our scripts simple, to allow users to easily wrap them in cgroups, containers, systemd units, etc. Besides, cgroups don't even exist on all supported platforms. We should not be baking in cgroups management into the scripts directly. The easier we make Accumulo

Re: Updates to Accumulo scripts for 2.0

2016-10-06 Thread Mike Miller
+1 I like the simplicity of the bin directory and the clarity of only having the 4 main scripts. As someone still new to the project, I am a fan of clean up and simplification by any means possible. Granted, I don't have to maintain production clusters or distributions like CDH so I'd imagine

Updates to Accumulo scripts for 2.0

2016-10-06 Thread Mike Walch
Hi Everyone, I started work on on simplifying the Accumulo scripts and configuration files for the 2.0 release. Given that this change could impact many, I wanted to solicit feedback early in the process. Below is the JIRA issue for this work and a pull request with some initial code:

[GitHub] accumulo issue #161: ACCUMULO-4453 Remove constructor code duplication and s...

2016-10-06 Thread joshelser
Github user joshelser commented on the issue: https://github.com/apache/accumulo/pull/161 Oh, `mvn verify -Psunny` passed with this change for me locally. --- 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

[GitHub] accumulo issue #161: ACCUMULO-4453 Remove constructor code duplication and s...

2016-10-06 Thread joshelser
Github user joshelser commented on the issue: https://github.com/apache/accumulo/pull/161 @dlmarion do you have a moment to take a glance? --- 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 have

[GitHub] accumulo pull request #161: ACCUMULO-4453 Remove constructor code duplicatio...

2016-10-06 Thread joshelser
GitHub user joshelser opened a pull request: https://github.com/apache/accumulo/pull/161 ACCUMULO-4453 Remove constructor code duplication and spammy DEBUG lo… …g message You can merge this pull request into a Git repository by running: $ git pull

[GitHub] accumulo pull request #160: Simplify Accumulo scripts and config files

2016-10-06 Thread mikewalch
GitHub user mikewalch opened a pull request: https://github.com/apache/accumulo/pull/160 Simplify Accumulo scripts and config files See [JIRA issue](https://issues.apache.org/jira/browse/ACCUMULO-4490) for background. This pull request is not ready to be merged. I am