Github user milleruntime commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/292#discussion_r132972878
--- Diff:
core/src/main/java/org/apache/accumulo/core/client/impl/Table.java ---
@@ -16,24 +16,59 @@
*/
package org.apache.accumulo.core.cl
Github user milleruntime commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/292#discussion_r132999867
--- Diff:
core/src/main/java/org/apache/accumulo/core/client/impl/Table.java ---
@@ -16,24 +16,59 @@
*/
package org.apache.accumulo.core.cl
Github user milleruntime commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/292#discussion_r133002408
--- Diff:
core/src/main/java/org/apache/accumulo/core/client/impl/Table.java ---
@@ -16,24 +16,59 @@
*/
package org.apache.accumulo.core.cl
Github user milleruntime commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/292#discussion_r133002554
--- Diff:
core/src/main/java/org/apache/accumulo/core/client/impl/Table.java ---
@@ -16,24 +16,59 @@
*/
package org.apache.accumulo.core.cl
Github user ctubbsii commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/292#discussion_r133059580
--- Diff:
core/src/main/java/org/apache/accumulo/core/client/impl/Table.java ---
@@ -16,24 +16,59 @@
*/
package org.apache.accumulo.core.client
Github user ctubbsii commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/292#discussion_r133068592
--- Diff:
core/src/main/java/org/apache/accumulo/core/client/impl/Table.java ---
@@ -16,24 +16,59 @@
*/
package org.apache.accumulo.core.client
Github user ctubbsii commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/292#discussion_r133070150
--- Diff:
core/src/main/java/org/apache/accumulo/core/client/impl/Table.java ---
@@ -16,24 +16,59 @@
*/
package org.apache.accumulo.core.client
Github user ctubbsii commented on the issue:
https://github.com/apache/accumulo/pull/292
This whole thing might be easier to implement with
[`CacheBuilder.weakValues()`][1] from Guava. Eviction seems to occur if value
disappears. I **think** this will do what we want, and it should au
Github user ctubbsii commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/291#discussion_r133086237
--- Diff:
server/base/src/main/java/org/apache/accumulo/server/metrics/MetricsSystemHelper.java
---
@@ -16,20 +16,65 @@
*/
package org.apache.
Github user ctubbsii commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/291#discussion_r133086488
--- Diff:
server/base/src/main/java/org/apache/accumulo/server/metrics/MetricsSystemHelper.java
---
@@ -16,20 +16,65 @@
*/
package org.apache.
Github user ctubbsii commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/291#discussion_r133086736
--- Diff:
server/base/src/main/java/org/apache/accumulo/server/metrics/MetricsSystemHelper.java
---
@@ -16,20 +16,65 @@
*/
package org.apache.
11 matches
Mail list logo