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;
}
