[
https://issues.apache.org/jira/browse/HBASE-4388?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13133478#comment-13133478
]
[email protected] commented on HBASE-4388:
------------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2539/
-----------------------------------------------------------
(Updated 2011-10-22 20:59:14.138541)
Review request for hbase.
Changes
-------
Address Ted feedback (I don't include tgz file in this diff)
Summary
-------
Changes name of column in root that holds current version of the meta table
from info:metamigrated to info:v.
Current version of meta table is 0 (short). The absence of the column means
table has not been migrated.
Upped version on HRegionInfo and made its readFields able to deserialze the
older pre-0.92 HRegionInfo.
Change MetaReader#putToCatalogTable so it no longer takes a region name. Was
redundant (needed this method for migration).
Removed a bunch of MetaMigrationRemovingHTD. Not needed any more or was
duplicated code and I axed it.
Found a bug in that we didn't migrate info:splitA or info:splitB if present --
i.e. if we had offlined parent from split going on when migration was
happening, we'd fail.
Migration is more idempotenty now. We look at the HRI bytes and if older
version, we'll update (Removed the try to deserialize and catch exception
stuff).
Moved methods from HBaseTestingUtility and from Writables that had to do w/
migration exclusively, out and into MetaMigrationRemoveHTD migrating class (or
into migration tests) so migration code all in one place rather than spread
about (more to do)
Changed MetaReader#isMetaTable to MetaReader#isRootTableRow. Makes more sense
now (needed migrating).
Renamed TestMetaMigration as TestMetaMigrationRemovingHTD. Also, made this
test startup on top of a 0.90 hbase.rootdir to test it can migrate.
Added TestMigrationFrom090To092
This addresses bug hbase-4388.
https://issues.apache.org/jira/browse/hbase-4388
Diffs (updated)
-----
src/main/java/org/apache/hadoop/hbase/HConstants.java 144f526
src/main/java/org/apache/hadoop/hbase/HRegionInfo.java e950df3
src/main/java/org/apache/hadoop/hbase/catalog/MetaEditor.java 6387df9
src/main/java/org/apache/hadoop/hbase/catalog/MetaMigrationRemovingHTD.java
0c1d221
src/main/java/org/apache/hadoop/hbase/catalog/MetaReader.java f7f4d60
src/main/java/org/apache/hadoop/hbase/master/MasterFileSystem.java 91abb00
src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java 12dfbcd
src/main/java/org/apache/hadoop/hbase/util/Writables.java f595af5
src/test/java/org/apache/hadoop/hbase/HBaseTestingUtility.java 6af3d35
src/test/java/org/apache/hadoop/hbase/client/TestMetaMigration.java 6cbf88c
src/test/java/org/apache/hadoop/hbase/client/TestMetaMigrationRemovingHTD.java
PRE-CREATION
src/test/java/org/apache/hadoop/hbase/migration/TestMigrationFrom090To092.java
PRE-CREATION
Diff: https://reviews.apache.org/r/2539/diff
Testing
-------
Migration tests pass.
Thanks,
Michael
> Second start after migration from 90 to trunk crashes
> -----------------------------------------------------
>
> Key: HBASE-4388
> URL: https://issues.apache.org/jira/browse/HBASE-4388
> Project: HBase
> Issue Type: Bug
> Components: master, migration
> Affects Versions: 0.92.0
> Reporter: Todd Lipcon
> Assignee: stack
> Priority: Blocker
> Fix For: 0.92.0
>
> Attachments: 4388-v2.txt, 4388-v3.txt, 4388-v4.txt, 4388.txt,
> hbase-4388-root.dir.tgz, hbase-master-nase.log, meta.tgz
>
>
> I started a trunk cluster to upgrade from 90, inserted a ton of data, then
> did a clean shutdown. When I started again, I got the following exception:
> 11/09/13 12:29:09 INFO master.HMaster: Meta has HRI with HTDs. Updating meta
> now.
> 11/09/13 12:29:09 FATAL master.HMaster: Unhandled exception. Starting
> shutdown.
> java.lang.NegativeArraySizeException: -102
> at org.apache.hadoop.hbase.util.Bytes.readByteArray(Bytes.java:147)
> at
> org.apache.hadoop.hbase.HTableDescriptor.readFields(HTableDescriptor.java:606)
> at
> org.apache.hadoop.hbase.migration.HRegionInfo090x.readFields(HRegionInfo090x.java:641)
> at
> org.apache.hadoop.hbase.util.Writables.getWritable(Writables.java:133)
> at
> org.apache.hadoop.hbase.util.Writables.getWritable(Writables.java:103)
> at
> org.apache.hadoop.hbase.util.Writables.getHRegionInfoForMigration(Writables.java:228)
> at
> org.apache.hadoop.hbase.catalog.MetaEditor.getHRegionInfoForMigration(MetaEditor.java:350)
> at
> org.apache.hadoop.hbase.catalog.MetaEditor$1.visit(MetaEditor.java:273)
> at
> org.apache.hadoop.hbase.catalog.MetaReader.fullScan(MetaReader.java:633)
> at
> org.apache.hadoop.hbase.catalog.MetaReader.fullScan(MetaReader.java:255)
> at
> org.apache.hadoop.hbase.catalog.MetaReader.fullScan(MetaReader.java:235)
> at
> org.apache.hadoop.hbase.catalog.MetaEditor.updateMetaWithNewRegionInfo(MetaEditor.java:284)
> at
> org.apache.hadoop.hbase.catalog.MetaEditor.migrateRootAndMeta(MetaEditor.java:298)
> at
> org.apache.hadoop.hbase.master.HMaster.updateMetaWithNewHRI(HMaster.java:529)
> at
> org.apache.hadoop.hbase.master.HMaster.finishInitialization(HMaster.java:472)
> at org.apache.hadoop.hbase.master.HMaster.run(HMaster.java:309)
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
