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

stack commented on HBASE-10201:
-------------------------------

bq.  I think it is better to open another issue to handle the duplication.

OK.

bq.  getEarliestFlushTimeForAllStore should be public because TestIOFencing use 
it(which in another package). 

FYI, we mark these with @VisibleForTesting annotation..  I can do on commit.

bq. ....but I see lots of other similar methods declared as public...

Yeah, sorry about that; we ain't always consistent.... trying.

bq. Does this meet the requirement?

Yes. Out of interest, are you using the hbase formatter?

bq. I tried but failed to make dev-support/test-patch.sh work properly...

Yeah, this stuff is focused on the master.  Unit tests passing on branch-1 
would be great. Just note it here in the issue.

You going to try hbase-it?

Thanks.

> Port 'Make flush decisions per column family' to trunk
> ------------------------------------------------------
>
>                 Key: HBASE-10201
>                 URL: https://issues.apache.org/jira/browse/HBASE-10201
>             Project: HBase
>          Issue Type: Improvement
>          Components: wal
>            Reporter: Ted Yu
>            Assignee: zhangduo
>            Priority: Critical
>             Fix For: 2.0.0, 0.99.2
>
>         Attachments: 3149-trunk-v1.txt, HBASE-10201-0.98.patch, 
> HBASE-10201-0.98_1.patch, HBASE-10201-0.98_2.patch, HBASE-10201-0.99.patch, 
> HBASE-10201.patch, HBASE-10201_1.patch, HBASE-10201_2.patch, 
> HBASE-10201_3.patch
>
>
> Currently the flush decision is made using the aggregate size of all column 
> families. When large and small column families co-exist, this causes many 
> small flushes of the smaller CF. We need to make per-CF flush decisions.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to