Review Board 1.7.22

hbase-3872 Hole in split transaction rollback; edits to .META. need to be rolled back even if it seems like they didn't make it

Review Request #1097 - Created July 13, 2011 and updated

Michael Stack
Fix is two-fold.

1. Fix catalogjanitor so that it does not remove parent if daughter regions are not present.
2. Fix the SplitTransaction so that if we go past the point of no return, DO NOT REMOVE daughter regions as part of cleanup; leave them in place.  Because we went past PONR, we are going to abort and server shutdown processing has what it needs to do fixup.
Added two new unit tests.  One to test we do not remove parent if no daughter region in fs and another to test that the right answer pops out of the call to rollback if we go past PONR (and that the daughter regions are still in place after rollback).
Posted (July 13, 2011, 10:13 p.m.)
There are a few white spaces.
I assume this is a copy-and-paste issue.
Posted (July 13, 2011, 10:49 p.m.)


Good one.  You +1 on committing?
Ship it!
Posted (July 13, 2011, 10:57 p.m.)