hbase-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ryan Rawson" <ryano...@gmail.com>
Subject Review Request: HBASE-2863 -- HBASE-2553 removed an important edge case
Date Thu, 22 Jul 2010 08:07:55 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for hbase.


There are tricky edge cases that were removed by HBASE-2553 (oopsy!)... flaky tests have been
illustrating them. This patch fixes those flaky tests to be not flaky (using the EnvironmentEdgeManager
thing) and also fixes them, and introduces tests that cover the particular use cases slightly
better as well. Oh yes and and fixes the actual bug.

Without these fixes we would end up with KVs with different values with the same Timestamp
which causes problems.  This can happen when we get more than 1 increment/millisecond and
especially during a snapshot.

This addresses bug HBASE-2863.


  src/main/java/org/apache/hadoop/hbase/KeyValue.java e32d683 
  src/main/java/org/apache/hadoop/hbase/regionserver/MemStore.java 44fa0c3 
  src/main/java/org/apache/hadoop/hbase/regionserver/Store.java c1ff9f2 
  src/main/java/org/apache/hadoop/hbase/util/ManualEnvironmentEdge.java PRE-CREATION 
  src/test/java/org/apache/hadoop/hbase/regionserver/TestHRegion.java 4ead02d 
  src/test/java/org/apache/hadoop/hbase/regionserver/TestStore.java a32eed6 

Diff: http://review.hbase.org/r/354/diff




View raw message