You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@accumulo.apache.org by ke...@deenlo.com on 2015/01/06 20:12:17 UTC
Re: Review Request 29176: ACCUMULO-3420 Get Visibility Metrics from
PrintInfo
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29176/#review66882
-----------------------------------------------------------
core/src/test/java/org/apache/accumulo/core/file/rfile/RFileMetricsTest.java
<https://reviews.apache.org/r/29176/#comment110579>
Is this copied from the RFile unit test? If can the two test be modified to share this code?
- kturner
On Dec. 22, 2014, 3:25 p.m., Jenna Huston wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/29176/
> -----------------------------------------------------------
>
> (Updated Dec. 22, 2014, 3:25 p.m.)
>
>
> Review request for accumulo.
>
>
> Bugs: ACCUMULO-3420
> https://issues.apache.org/jira/browse/ACCUMULO-3420
>
>
> Repository: accumulo
>
>
> Description
> -------
>
> Added an option to PrintInfo to get visibility metrics. Prints the number of times a visibilty is seen in each locality group. Also shows how many blocks in the locality group have his visibiltiy.
>
>
> Diffs
> -----
>
> core/src/main/java/org/apache/accumulo/core/file/rfile/MetricsGatherer.java PRE-CREATION
> core/src/main/java/org/apache/accumulo/core/file/rfile/PrintInfo.java 43586dd
> core/src/main/java/org/apache/accumulo/core/file/rfile/RFile.java 9dcb3a5
> core/src/main/java/org/apache/accumulo/core/file/rfile/VisMetricsGatherer.java PRE-CREATION
> core/src/test/java/org/apache/accumulo/core/file/rfile/RFileMetricsTest.java PRE-CREATION
>
> Diff: https://reviews.apache.org/r/29176/diff/
>
>
> Testing
> -------
>
> Tested with a few RFiles that I made.
>
>
> Thanks,
>
> Jenna Huston
>
>