[GitHub] accumulo pull request #224: ACCUMULO-4500 ACCUMULO-96 Added summarization

2017-03-19 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/224#discussion_r106811693 --- Diff: server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServerResourceManager.java --- @@ -232,7 +234,9 @@ public

[GitHub] accumulo pull request #224: ACCUMULO-4500 ACCUMULO-96 Added summarization

2017-03-19 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/224#discussion_r106811717 --- Diff: test/src/main/java/org/apache/accumulo/test/functional/SummaryIT.java --- @@ -770,4 +774,44 @@ public void testExceptions() throws Exception {

[GitHub] accumulo pull request #224: ACCUMULO-4500 ACCUMULO-96 Added summarization

2017-03-19 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/224#discussion_r106811796 --- Diff: core/src/main/java/org/apache/accumulo/core/client/summary/Summary.java --- @@ -78,27 +80,38 @@ public long getLarge() { }

[GitHub] accumulo pull request #224: ACCUMULO-4500 ACCUMULO-96 Added summarization

2017-03-19 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/224#discussion_r106811743 --- Diff: core/src/main/java/org/apache/accumulo/core/client/admin/TableOperations.java --- @@ -835,7 +835,7 @@ void setSamplerConfiguration(String

[GitHub] accumulo pull request #224: ACCUMULO-4500 ACCUMULO-96 Added summarization

2017-03-19 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/224#discussion_r106811664 --- Diff: server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServer.java --- @@ -1812,7 +1884,26 @@ public TSummaries

[GitHub] accumulo pull request #224: ACCUMULO-4500 ACCUMULO-96 Added summarization

2017-03-19 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/224#discussion_r106811464 --- Diff: core/src/main/java/org/apache/accumulo/core/conf/Property.java --- @@ -362,8 +362,14 @@ "The number of threads available to load