hbase-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alex Newman" <newa...@cloudera.com>
Subject Re: Review Request: Refactor "Corrupt Data" Tests in TestHLogSplit
Date Mon, 01 Nov 2010 17:04:09 GMT


> On 2010-10-31 21:18:46, Nicolas wrote:
> > I'm okay with the general concept/direction.  I think you should get rid of resetLogReaderClass(),
but other than that, it looks pretty clean :)  I think there's more edge cases with IOE/ParseExceptions
that we could theoretically handle, but this is a positive step and mitigates the regression.

Right on, what are some edge cases? What's the best way to nuke that state? Eventually, with
distributed log splitting, this will eventually need to work in a distributed way.


- Alex


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


On 2010-10-29 15:03:31, Alex Newman wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://review.cloudera.org/r/1115/
> -----------------------------------------------------------
> 
> (Updated 2010-10-29 15:03:31)
> 
> 
> Review request for hbase and Nicolas.
> 
> 
> Summary
> -------
> 
> So I took the approach taken with the instrumented sequence file rider. Not sure if exposing
those methods in the WALReader is cool so I am down with other suggestions.
> 
> - Add a FaultySequenceFileLogReader
> - Un-commented the 2935 tests
> 
> 
> This addresses bug hbase-2935.
>     http://issues.apache.org/jira/browse/hbase-2935
> 
> 
> Diffs
> -----
> 
>   src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLog.java c8a10af 
>   src/main/java/org/apache/hadoop/hbase/regionserver/wal/SequenceFileLogReader.java 0b7dc78

>   src/test/java/org/apache/hadoop/hbase/regionserver/wal/FaultySequenceFileLogReader.java
PRE-CREATION 
>   src/test/java/org/apache/hadoop/hbase/regionserver/wal/TestHLogSplit.java 473c359 
> 
> Diff: http://review.cloudera.org/r/1115/diff
> 
> 
> Testing
> -------
> 
> Running on our internal hudson as I type this. Also I ran the TestHLogSplit test locally.
I'll update the review if hudson is green.
> 
> 
> Thanks,
> 
> Alex
> 
>


Mime
View raw message