[
https://issues.apache.org/jira/browse/HBASE-451?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13054714#comment-13054714
]
Gary Helmling commented on HBASE-451:
-------------------------------------
@Ted, good digging.
I'm not very familiar with these changes, but it looks to me like the changes
so far have tried to pull HTableDescription handling out of HRegion. So adding
it back in to HRegion.createHRegion() may be a step back.
I think I'd opt for trying to fix the tests to call
FSUtils.createTableDescriptor() instead. Either in TestTableMapReduce.<init>
or MultiRegionTable.preHBaseClusterSetup(), prior to creating the table
regions. I think either of those would work. I wonder how many other
HBaseTestCase subclasses may have problems as well.
> Remove HTableDescriptor from HRegionInfo
> ----------------------------------------
>
> Key: HBASE-451
> URL: https://issues.apache.org/jira/browse/HBASE-451
> Project: HBase
> Issue Type: Improvement
> Components: master, regionserver
> Affects Versions: 0.2.0
> Reporter: Jim Kellerman
> Assignee: Subbu M Iyer
> Priority: Critical
> Fix For: 0.92.0
>
> Attachments: 451-addendum.txt,
> 451_support_for_removing_HTD_from_HRI_trunk.txt,
> HBASE-451-Fixed_broken_TestAdmin.patch,
> HBASE-451-Fixed_broken_TestAdmin1.patch,
> HBASE-451_-_First_draft_support_for_removing_HTD_from_HRI1.patch,
> HBASE-451_-_Fourth_draft_support_for_removing_HTD_from_HRI.patch,
> HBASE-451_-_Second_draft_-_Remove_HTD_from_HRI.patch, descriptors.txt,
> fixtestadmin.txt, pass_htd_on_region_construction.txt
>
>
> There is an HRegionInfo for every region in HBase. Currently HRegionInfo also
> contains the HTableDescriptor (the schema). That means we store the schema n
> times where n is the number of regions in the table.
> Additionally, for every region of the same table that the region server has
> open, there is a copy of the schema. Thus it is stored in memory once for
> each open region.
> If HRegionInfo merely contained the table name the HTableDescriptor could be
> stored in a separate file and easily found.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
