[ https://issues.apache.org/jira/browse/HBASE-5801?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13261969#comment-13261969 ]
jirapos...@reviews.apache.org commented on HBASE-5801: ------------------------------------------------------ ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4833/#review7237 ----------------------------------------------------------- Ship it! lgtm. - jmhsieh On 2012-04-23 20:16:23, Jimmy Xiang wrote: bq. bq. ----------------------------------------------------------- bq. This is an automatically generated e-mail. To reply, visit: bq. https://reviews.apache.org/r/4833/ bq. ----------------------------------------------------------- bq. bq. (Updated 2012-04-23 20:16:23) bq. bq. bq. Review request for hbase and jmhsieh. bq. bq. bq. Summary bq. ------- bq. bq. Added option to fix inconsistent table descriptors: bq. 1. sideline the current .regioninfo file bq. 2. create a new one with HTD from HBaseAdmin (meta, first entry) bq. 3. offline the region and wait till it assigned again bq. bq. bq. This addresses bug HBASE-5801. bq. https://issues.apache.org/jira/browse/HBASE-5801 bq. bq. bq. Diffs bq. ----- bq. bq. src/main/java/org/apache/hadoop/hbase/util/HBaseFsck.java 50f9128 bq. src/main/java/org/apache/hadoop/hbase/util/HBaseFsckRepair.java 06d2b73 bq. src/test/java/org/apache/hadoop/hbase/util/TestHBaseFsck.java 103d8bf bq. bq. Diff: https://reviews.apache.org/r/4833/diff bq. bq. bq. Testing bq. ------- bq. bq. TestHBaseFsck* are green. On live cluster, it does the fix as expected. bq. bq. bq. Thanks, bq. bq. Jimmy bq. bq. > [hbck] Hbck should handle case where some regions have different HTD settings > in .regioninfo files (0.90 specific) > ------------------------------------------------------------------------------------------------------------------- > > Key: HBASE-5801 > URL: https://issues.apache.org/jira/browse/HBASE-5801 > Project: HBase > Issue Type: Improvement > Components: hbck > Affects Versions: 0.90.7 > Reporter: Jonathan Hsieh > Assignee: Jimmy Xiang > Fix For: 0.90.7 > > Attachments: hbase_5801_v2.patch > > > Recently, we encountered a case where some regions in a table have different > HTableDescriptor settings serialized into HDFS their HRegionInfo .regioninfo > file. hbck expects all HTDs within a table to be the same and currently > bails out in this situation. > We need to either point out a proper set of actions for the user to execute > or automatically convert the region to a common HTD (likely the most common > on, or possibly the "first one".) > Not sure if this requires reformatting data but may require closing and > restarting a region. > This issue is hbase 0.90.x specific -- 0.92+ keep all table info in a single > .tableinfo file. -- 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