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

Jim Kellerman commented on HBASE-467:
-------------------------------------

This is looking ugly. HRegion references a number of methods in HStore and 
HStoreFile.
HStore references static methods from HRegion.

HRegion and HStore are also tightly coupled with HLog.

We can probably factor out the inner classes of Hregion, HStore and HStoreFile 
into
o.a.h.h.regionserver.{region,store,storefile(??)}, but trying to tease these 
apart using the current plan will either be a) a ton of work or b) turn out to 
not be possible.

Comments, please!

> Move HStore(^Key)*.java into o.a.h.h.store
> ------------------------------------------
>
>                 Key: HBASE-467
>                 URL: https://issues.apache.org/jira/browse/HBASE-467
>             Project: Hadoop HBase
>          Issue Type: Sub-task
>            Reporter: Bryan Duxbury
>            Assignee: Jim Kellerman
>            Priority: Minor
>
> Per Jim's suggestions on HBase-419, let's move all the store-related files 
> into a subpackage. It should either be o.a.h.h.store or 
> o.a.h.h.regionserver.store. If we push it down another level, I think that we 
> should make sure we never use any of those files outside of 
> o.a.h.h.regionserver.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to