svn commit: r1181372 - /hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFile.java

[ns...@apache.org]

Author: nspiegelberg
Date: Tue Oct 11 02:02:52 2011
New Revision: 1181372

URL: http://svn.apache.org/viewvc?rev=1181372&view=rev
Log:
Fixing sequenceid problem in StoreFile

Summary:
There was a bug in StoreFile wherein we used to throw an Exception when the
sequence id was -1. However, the bulk load files did not have a sequence id and
they had to use the default sequence id of -1. In such cases, any query
getSequenceID() would result in throwing an Exception.

This was fixed in public trunk by Stack as part of a bigger change in revision
991397 for HBASE-2692 on 8/31/10.

Test Plan:
The test which was failing previously is running properly after using this
patch. Also running the entire test suite once again.

DiffCamp Revision: 156055
Reviewed By: jgray
CC: jgray, hbase@lists
Revert Plan:
OK

Modified:
    hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFile.java

Modified: hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFile.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFile.java?rev=1181372&r1=1181371&r2=1181372&view=diff
==============================================================================
--- hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFile.java (original)
+++ hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFile.java Tue Oct 11 02:02:52 2011
@@ -291,9 +291,6 @@ public class StoreFile {
    * @return This files maximum edit sequence id.
    */
   public long getMaxSequenceId() {
-    if (this.sequenceid == -1) {
-      throw new IllegalAccessError("Has not been initialized");
-    }
     return this.sequenceid;
   }