[ 
https://issues.apache.org/jira/browse/HBASE-7763?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13574942#comment-13574942
 ] 

Ted Yu commented on HBASE-7763:
-------------------------------

For PerfTestCompactionPolicy:
{code}
+  public static Collection<Object[]> data() {
+    return Arrays.asList(new Object[][] {
+        {"Default",   new DefaultCompactionPolicy(), 3, 2, 1.2f},
+        {"Default",   new DefaultCompactionPolicy(), 4, 2, 1.2f},
+        {"Default",   new DefaultCompactionPolicy(), 5, 2, 1.2f},
+        {"Default",   new DefaultCompactionPolicy(), 4, 2, 1.3f},
+        {"Default",   new DefaultCompactionPolicy(), 4, 2, 1.4f},
{code}
I think other compaction policies would be added to the above list. Should the 
test be named PerfTestCompactionPolicies ?
{code}
+  public PerfTestCompactionPolicy(String name, CompactionPolicy cp, int max, 
int min, float ratio) {
{code}
Please add javadoc for the parameters.
{code}
+  static final Log LOG = LogFactory.getLog(PerfTestCompactionPolicy.class);
...
+    //print out tab delimited so that it can be used in excel/gdocs.
+    System.out.println(
+                     name
{code}
Should LOG be used above ?
                
> Compactions not sorting based on size anymore.
> ----------------------------------------------
>
>                 Key: HBASE-7763
>                 URL: https://issues.apache.org/jira/browse/HBASE-7763
>             Project: HBase
>          Issue Type: Bug
>          Components: Compaction
>    Affects Versions: 0.96.0, 0.94.4
>            Reporter: Elliott Clark
>            Assignee: Elliott Clark
>            Priority: Critical
>             Fix For: 0.96.0, 0.94.6
>
>         Attachments: HBASE-7763-trunk-1.patch, HBASE-7763-trunk-2.patch, 
> HBASE-7763-trunk-TESTING.patch, HBASE-7763-trunk-TESTING.patch, 
> HBASE-7763-trunk-TESTING.patch
>
>
> Currently compaction selection is not sorting based on size.  This causes 
> selection to choose larger files to re-write than are needed when bulk loads 
> are involved.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to