Mailing-List: contact dev-help@hbase.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@hbase.apache.org
Received-SPF: neutral (nike.apache.org: local policy)
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Subject: Review Request: Improvements to HBaseFsck tool
From: "Todd Lipcon" <todd@cloudera.com>
To: "Kannan Muthukkaruppan" <kannan@facebook.com>,
	"dhruba borthakur" <dhruba@gmail.com>, stack@duboce.net
Date: Wed, 28 Jul 2010 23:22:47 -0000
Message-ID: <20100728232247.12819.37646@ip-10-202-7-187.ec2.internal>
Cc: "Todd Lipcon" <todd@cloudera.com>, dev@hbase.apache.org,
	jiraposter@review.hbase.org


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://review.cloudera.org/r/418/
-----------------------------------------------------------

Review request for hbase, stack, dhruba borthakur, and Kannan Muthukkaruppa=
n.


Summary
-------

Bunch of changes to the HBaseFsck tool to detect a wider variety of issues =
and to make it easier to add "fix" support for these problems in the future.


This addresses bug HBASE-2887.
    http://issues.apache.org/jira/browse/HBASE-2887


Diffs
-----

  src/main/java/org/apache/hadoop/hbase/client/HBaseFsck.java 19cdec1 =

  src/main/java/org/apache/hadoop/hbase/client/MetaScanner.java 5f8b909 =

  src/test/java/org/apache/hadoop/hbase/client/TestHBaseFsck.java PRE-CREAT=
ION =


Diff: http://review.cloudera.org/r/418/diff


Testing
-------

Tested an earlier variation of this patch against my cluster in various scr=
ewed up states. This patch adds a basic unit test but it could be improved =
to use a mock cluster and test more variations.


Thanks,

Todd