[GitHub] accumulo pull request: ACCUMULO-4145 Eliminate Text wrapping of ta...

2016-02-18 Thread ctubbsii
GitHub user ctubbsii opened a pull request: https://github.com/apache/accumulo/pull/72 ACCUMULO-4145 Eliminate Text wrapping of tableIDs * Best attempt to eliminate text wrapping of tableIDs on master branch. * Some ByteBuffer wrapping still occurs due to thrift using bytes to

[GitHub] accumulo pull request: ACCUMULO-4145 Eliminate Text wrapping of ta...

2016-02-18 Thread ctubbsii
Github user ctubbsii closed the pull request at: https://github.com/apache/accumulo/pull/70 --- 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 this feature enabled and wishes so, or if the feature is

Re: [DISCUSS] Pulling out the examples

2016-02-18 Thread Tony Kurc
> > > A PMC governs a TLP. A TLP has >= 1 repository of code. > > nifi has 3.

Re: [DISCUSS] Pulling out the examples

2016-02-18 Thread Josh Elser
Thanks for the quick answers already, Christopher. I don't necessarily want to say those were rhetorical questions, but I was hoping to see a document put together which outlines these concerns (I know Keith has a been a strong proponent against bit-rot in the past, and assumed he's throught

Re: [DISCUSS] Pulling out the examples

2016-02-18 Thread Christopher
On Thu, Feb 18, 2016 at 7:37 PM Josh Elser wrote: > I'd like to see some process put into place to mitigate "bit-rot". If > the examples don't live in the "main" repository, how do we make sure > they don't get ignored and become dead or "bad" code? > > For questions at the

Re: [DISCUSS] Pulling out the examples

2016-02-18 Thread Josh Elser
Christopher wrote: I also wonder what the ASF rule-of-thumb is for stuff like this... the examples aren't really "projects" in the sense that they are moving towards an official ASF "release", so much as they are code serving as runnable documentation. I do have the concern that every

[GitHub] accumulo pull request: ACCUMULO-4138 Fix the description of -b opt...

2016-02-18 Thread joshelser
Github user joshelser commented on the pull request: https://github.com/apache/accumulo/pull/71#issuecomment-185994820 > I was able to merge this PR into 1.7 with no conflicts and then merge the result of 1.7+PR into master with no conflicts. If that is not the case, let me know and

Re: [DISCUSS] Pulling out the examples

2016-02-18 Thread Christopher
I don't know what it means to be a "formal subproject", but I'm +1 for disconnecting the examples from our releases, and putting them in their own repo to assist in making that happen. I'm not sure what to do about wikisearch. It should probably go in there, too, but what about its existing repo?

Re: [DISCUSS] Pulling out the examples

2016-02-18 Thread Josh Elser
I'd like to see some process put into place to mitigate "bit-rot". If the examples don't live in the "main" repository, how do we make sure they don't get ignored and become dead or "bad" code? For questions at the foreground now: * Can we set up new CI jobs that build the new examples repo

Re: [DISCUSS] Pulling out the examples

2016-02-18 Thread Sean Busbey
I'd be +1 to making them an independent repo still controlled by the PMC. Whether they need to be a formal subproject or not I think could go either way. -- Sean Busbey On Feb 18, 2016 14:57, "Michael Wall" wrote: > Talking with Keith and Christopher today, they mentioned it

[GitHub] accumulo pull request: ACCUMULO-4138 Fix the description of -b opt...

2016-02-18 Thread ctubbsii
Github user ctubbsii commented on a diff in the pull request: https://github.com/apache/accumulo/pull/71#discussion_r53393673 --- Diff: core/src/main/java/org/apache/accumulo/core/util/shell/commands/OptUtil.java --- @@ -117,7 +117,7 @@ public static OptionGroup

[GitHub] accumulo pull request: ACCUMULO-4138 Fix the description of -b opt...

2016-02-18 Thread mjwall
Github user mjwall commented on the pull request: https://github.com/apache/accumulo/pull/71#issuecomment-185954621 Going to let this sit until tomorrow and then I will update the PR with (exclusive). I was able to merge this PR into 1.7 with no conflicts and then merge the

[GitHub] accumulo pull request: ACCUMULO-4138 Fix the description of -b opt...

2016-02-18 Thread mjwall
Github user mjwall commented on a diff in the pull request: https://github.com/apache/accumulo/pull/71#discussion_r53391009 --- Diff: core/src/main/java/org/apache/accumulo/core/util/shell/commands/OptUtil.java --- @@ -117,7 +117,7 @@ public static OptionGroup

[DISCUSS] Pulling out the examples

2016-02-18 Thread Michael Wall
Talking with Keith and Christopher today, they mentioned it might be useful to pull the examples out of Accumulo into a separate project with it's own repo. Here are some talking points - Provide an easy means to run examples against the MiniAccumuloCluster. - Give users the ability to run

[GitHub] accumulo pull request: ACCUMULO-4138 Fix the description of -b opt...

2016-02-18 Thread mjwall
GitHub user mjwall opened a pull request: https://github.com/apache/accumulo/pull/71 ACCUMULO-4138 Fix the description of -b options It was wrong for FlushCommand and CompactCommand. In the process of refactoring, the following was accomplished: - Change the