[GitHub] accumulo pull request #279: ACCUMULO-3238 Table.ID Namespace.ID Refactor

2017-07-17 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/accumulo/pull/279#discussion_r127780058 --- Diff: core/src/main/java/org/apache/accumulo/core/client/impl/AbstractId.java --- @@ -0,0 +1,86 @@ +/* + * Licensed to the Apache Software

[GitHub] accumulo pull request #275: ACCUMULO-4667 Reworked the LocalityGroupIterator...

2017-07-17 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/accumulo/pull/275#discussion_r127782947 --- Diff: core/src/main/java/org/apache/accumulo/core/iterators/system/LocalityGroupIterator.java --- @@ -97,75 +134,122 @@ public static final int s

[GitHub] accumulo pull request #279: ACCUMULO-3238 Table.ID Namespace.ID Refactor

2017-07-17 Thread milleruntime
Github user milleruntime commented on a diff in the pull request: https://github.com/apache/accumulo/pull/279#discussion_r127793774 --- Diff: core/src/main/java/org/apache/accumulo/core/client/impl/AbstractId.java --- @@ -0,0 +1,86 @@ +/* + * Licensed to the Apache Software

[GitHub] accumulo pull request #275: ACCUMULO-4667 Reworked the LocalityGroupIterator...

2017-07-17 Thread ivakegg
Github user ivakegg commented on a diff in the pull request: https://github.com/apache/accumulo/pull/275#discussion_r127797105 --- Diff: core/src/main/java/org/apache/accumulo/core/util/LocalityGroupUtil.java --- @@ -50,16 +52,16 @@ // private static final Logger log =

[GitHub] accumulo pull request #275: ACCUMULO-4667 Reworked the LocalityGroupIterator...

2017-07-17 Thread ivakegg
Github user ivakegg commented on a diff in the pull request: https://github.com/apache/accumulo/pull/275#discussion_r127797290 --- Diff: core/src/main/java/org/apache/accumulo/core/iterators/system/LocalityGroupIterator.java --- @@ -97,75 +134,122 @@ public static final int seek(H

[GitHub] accumulo pull request #275: ACCUMULO-4667 Reworked the LocalityGroupIterator...

2017-07-17 Thread ivakegg
Github user ivakegg commented on a diff in the pull request: https://github.com/apache/accumulo/pull/275#discussion_r127797510 --- Diff: core/src/main/java/org/apache/accumulo/core/util/LocalityGroupUtil.java --- @@ -50,16 +52,16 @@ // private static final Logger log =

[GitHub] accumulo pull request #275: ACCUMULO-4667 Reworked the LocalityGroupIterator...

2017-07-17 Thread ivakegg
Github user ivakegg commented on a diff in the pull request: https://github.com/apache/accumulo/pull/275#discussion_r127797716 --- Diff: core/src/main/java/org/apache/accumulo/core/iterators/system/LocalityGroupIterator.java --- @@ -97,75 +134,122 @@ public static final int seek(H

[GitHub] accumulo pull request #275: ACCUMULO-4667 Reworked the LocalityGroupIterator...

2017-07-17 Thread ivakegg
Github user ivakegg commented on a diff in the pull request: https://github.com/apache/accumulo/pull/275#discussion_r127797989 --- Diff: core/src/main/java/org/apache/accumulo/core/iterators/system/LocalityGroupIterator.java --- @@ -16,23 +16,28 @@ */ package org.apache

[GitHub] accumulo pull request #275: ACCUMULO-4667 Reworked the LocalityGroupIterator...

2017-07-17 Thread ivakegg
Github user ivakegg commented on a diff in the pull request: https://github.com/apache/accumulo/pull/275#discussion_r127798040 --- Diff: core/src/main/java/org/apache/accumulo/core/iterators/system/LocalityGroupIterator.java --- @@ -65,23 +70,55 @@ public InterruptibleIterator get

[GitHub] accumulo pull request #275: ACCUMULO-4667 Reworked the LocalityGroupIterator...

2017-07-17 Thread ivakegg
Github user ivakegg commented on a diff in the pull request: https://github.com/apache/accumulo/pull/275#discussion_r127798094 --- Diff: core/src/main/java/org/apache/accumulo/core/iterators/system/LocalityGroupIterator.java --- @@ -97,75 +134,122 @@ public static final int seek(H

[GitHub] accumulo issue #200: Accumulo 4558 Added shell command to display server sta...

2017-07-17 Thread lstav
Github user lstav commented on the issue: https://github.com/apache/accumulo/pull/200 I just fixed the conflict with the main branch (it was due to the change being done to the old monitor code), I spoke with @milleruntime offline and he might take over fixing what had been discussed