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

Jimmy Xiang commented on HBASE-7412:
------------------------------------

Making HTableDescriptor take in a Conf in constructor will make the interface 
cleaner.  However, it involves other changes.  We still need to change the code 
calling #getMaxFileSize and #getMemStoreFlushSize since the existing code is 
wrong assuming the default setting must not be from the table specific setting. 
Let me think about it how we fix this.

I will fix the javadoc errors.

Yes, the un-boxing code is not good.  It is some existing code.  I will fix it.
                
> Fix how HTableDescriptor handles default max file size and flush size
> ---------------------------------------------------------------------
>
>                 Key: HBASE-7412
>                 URL: https://issues.apache.org/jira/browse/HBASE-7412
>             Project: HBase
>          Issue Type: Bug
>          Components: regionserver
>            Reporter: Jimmy Xiang
>            Assignee: Jimmy Xiang
>            Priority: Minor
>             Fix For: 0.96.0, 0.94.4
>
>         Attachments: 0.94-7412.patch, trunk-7412.patch
>
>
> If the region flush size is not set in the table, 
> IncreasingToUpperBoundRegionSplitPolicy will most likely always use the 
> default value: 128MB, even if the flush size is set to a different value in 
> hbase-site.xml.

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