[ https://issues.apache.org/jira/browse/HBASE-4322?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13104590#comment-13104590 ]
jirapos...@reviews.apache.org commented on HBASE-4322: ------------------------------------------------------ bq. On 2011-09-13 04:19:45, Michael Stack wrote: bq. > src/main/java/org/apache/hadoop/hbase/util/HBaseFsck.java, line 932 bq. > <https://reviews.apache.org/r/1789/diff/2/?file=40218#file40218line932> bq. > bq. > This comparator works properly w/ the null start and end keys? bq. bq. jmhsieh wrote: bq. I this is a problem. Ok if I address it in HBASE-4379? bq. bq. Michael Stack wrote: bq. Sure, but does that mean I can't apply this till hbase-4379 is done? Ok, I'll fix it here. Looks like the HRegionInfo comparator has the same problem (I'll file a separate issue for that). - jmhsieh ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1789/#review1871 ----------------------------------------------------------- On 2011-09-13 01:56:43, jmhsieh wrote: bq. bq. ----------------------------------------------------------- bq. This is an automatically generated e-mail. To reply, visit: bq. https://reviews.apache.org/r/1789/ bq. ----------------------------------------------------------- bq. bq. (Updated 2011-09-13 01:56:43) bq. bq. bq. Review request for hbase, Todd Lipcon and Michael Stack. bq. bq. bq. Summary bq. ------- bq. bq. This requires HBASE-4321. bq. bq. commit 0ed6be1b628f3c030480164af37825702a2d5020 bq. Author: Jonathan Hsieh <j...@cloudera.com> bq. Date: Mon Sep 5 02:16:32 2011 -0700 bq. bq. HBASE-4322 [hbck] Update checkIntegrity/checkRegionChain to present more accurate region split problem summary bq. bq. Minor semantics change bq. - if two regions overlap this counts as two inconsistencies instead of one. If three overlap, it counts as three. bq. - testMetaHole was previously actually was an overlap ([D,E] overlaps [C,]). Change to make true hole. bq. bq. bq. This addresses bug HBASE-4322. bq. https://issues.apache.org/jira/browse/HBASE-4322 bq. bq. bq. Diffs bq. ----- bq. bq. src/main/java/org/apache/hadoop/hbase/util/HBaseFsck.java 1a704f6 bq. src/test/java/org/apache/hadoop/hbase/util/TestHBaseFsck.java f42a87c bq. bq. Diff: https://reviews.apache.org/r/1789/diff bq. bq. bq. Testing bq. ------- bq. bq. Modified unit test passes. Currently running full suite. bq. bq. bq. Thanks, bq. bq. jmhsieh bq. bq. > [hbck] Update checkIntegrity/checkRegionChain to present more accurate region > split problem summary > --------------------------------------------------------------------------------------------------- > > Key: HBASE-4322 > URL: https://issues.apache.org/jira/browse/HBASE-4322 > Project: HBase > Issue Type: Improvement > Components: hbck > Affects Versions: 0.90.4, 0.94.0 > Reporter: Jonathan Hsieh > Assignee: Jonathan Hsieh > Attachments: > 0001-HBASE-4322-hbck-Update-checkIntegrity-checkRegionCha.patch, > 0001-HBASE-4322-hbck-Update-checkIntegrity-checkRegionCha.patch, > hbase-4322-0.90.patch > > > This is a mostly semantics preserving upgrade to hbck that uses the > RegionSplitCalculator from HBASE-4321 that provides more in depth information > about region split problems in meta when running hbck. -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira