----------------------------------------------------------- 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 > >