Re: [VOTE] Apache Accumulo 1.6.1 RC1

2014-09-24 Thread Keith Turner
Rescinding my -1 vote. Josh helped me figure this one out on IRC. There used to be a tag named 1.6.1-rc1. That no longer exists at apache repo, now there is a branch called 1.6.1-rc. I had both the old tag and the new branch, git was taking the tag. git fetch --prune did not remove the tag, I

Re: [VOTE] Apache Accumulo 1.6.1 RC1

2014-09-24 Thread Josh Elser
Accumulo server logs are ~118M. The CI logs are tiny. Hadoop logs I have are around 126M (although probably relevant bits are much smaller). Sadly, I didn't run with archived WALs, so I'm not sure how useful the server logs are on their own. Sean Busbey wrote: Josh, how big are all the

Re: [VOTE] Apache Accumulo 1.6.1 RC1

2014-09-24 Thread Keith Turner
+1 I did a little more poking at 1.6.1 and it looks good. Thanks Corey and Josh for putting these releases together. Sigs and hashes for bin.tar.gz and src.tar.gz look good Successfully ran mutslam[1] against 1.6.1 rc1, using staging repo. Ci w/ Agitation verify ran successfully. Env hadoop

Re: [VOTE] Apache Accumulo 1.6.1 RC1

2014-09-24 Thread Corey Nolet
Bill, I've been having that same IT issue and said the same thing It's not happening to others. I lifted the timeout completely and it never finished. On Wed, Sep 24, 2014 at 1:13 PM, Mike Drob mad...@cloudera.com wrote: Any chance the IRC chats can make it only the ML for posterity? Mike

[GitHub] accumulo pull request: ACCUMULO-3089: Create volume chooser from t...

2014-09-24 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/accumulo/pull/16#discussion_r17987075 --- Diff: core/src/main/java/org/apache/accumulo/core/client/admin/TableOperations.java --- @@ -103,6 +103,25 @@ void create(String tableName,

[GitHub] accumulo pull request: ACCUMULO-3089: Create volume chooser from t...

2014-09-24 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/accumulo/pull/16#discussion_r17987372 --- Diff: test/src/test/java/org/apache/accumulo/test/VolumeChooserIT.java --- @@ -0,0 +1,670 @@ +/* + * Licensed to the Apache Software

[GitHub] accumulo pull request: ACCUMULO-3089: Create volume chooser from t...

2014-09-24 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/accumulo/pull/16#discussion_r17987487 --- Diff: test/src/test/java/org/apache/accumulo/test/VolumeChooserIT.java --- @@ -0,0 +1,670 @@ +/* + * Licensed to the Apache Software

[GitHub] accumulo pull request: ACCUMULO-3089: Create volume chooser from t...

2014-09-24 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/accumulo/pull/16#discussion_r17987756 --- Diff: server/base/src/main/java/org/apache/accumulo/server/fs/VolumeChooser.java --- @@ -18,5 +18,7 @@ public interface

[GitHub] accumulo pull request: ACCUMULO-3089: Create volume chooser from t...

2014-09-24 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/accumulo/pull/16#discussion_r17987986 --- Diff: server/base/src/main/java/org/apache/accumulo/server/fs/VolumeManagerImpl.java --- @@ -560,8 +565,22 @@ public ContentSummary

[GitHub] accumulo pull request: ACCUMULO-3089: Create volume chooser from t...

2014-09-24 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/accumulo/pull/16#discussion_r17988361 --- Diff: test/src/test/java/org/apache/accumulo/test/VolumeChooserIT.java --- @@ -0,0 +1,670 @@ +/* + * Licensed to the Apache Software

[GitHub] accumulo pull request: ACCUMULO-3089: Create volume chooser from t...

2014-09-24 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/accumulo/pull/16#discussion_r17989515 --- Diff: server/base/src/main/java/org/apache/accumulo/server/fs/GeneralVolumeChooser.java --- @@ -0,0 +1,60 @@ +/* + * Licensed to the

[GitHub] accumulo pull request: ACCUMULO-3089: Create volume chooser from t...

2014-09-24 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/accumulo/pull/16#discussion_r17989615 --- Diff: core/src/main/java/org/apache/accumulo/core/client/admin/TableOperations.java --- @@ -103,6 +103,25 @@ void create(String tableName,

Thoughts on scaling back github integration

2014-09-24 Thread Mike Drob
Devs, Github integration is nice... but, I'm not a fan of how comments on diffs are posted to the JIRAs. Each line note is a separate comment and it includes a ton of irrelevant diff output. Looking at ACCUMULO-3089 is what triggered this reaction for me. Does anybody else have thoughts on this?

Re: Thoughts on scaling back github integration

2014-09-24 Thread Christopher
I was thinking about this... and I was wondering if this is done by the ASF integration could batch these comments for mailing to the dev list and/or commenting on JIRA. I really like using GitHub for reviews/comments. Issuing pull requests is very seamless, but the comment activity is a bit