You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ji...@apache.org on 2008/09/23 00:00:31 UTC

svn commit: r698016 - in /hadoop/hbase/trunk: CHANGES.txt src/java/org/apache/hadoop/hbase/io/Cell.java

Author: jimk
Date: Mon Sep 22 15:00:31 2008
New Revision: 698016

URL: http://svn.apache.org/viewvc?rev=698016&view=rev
Log:
HBASE-892   Cell iteration is broken (Doğacan Güney via Jim Kellerman)

Modified:
    hadoop/hbase/trunk/CHANGES.txt
    hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/io/Cell.java

Modified: hadoop/hbase/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/hbase/trunk/CHANGES.txt?rev=698016&r1=698015&r2=698016&view=diff
==============================================================================
--- hadoop/hbase/trunk/CHANGES.txt (original)
+++ hadoop/hbase/trunk/CHANGES.txt Mon Sep 22 15:00:31 2008
@@ -4,6 +4,7 @@
 
   BUG FIXES
    HBASE-891   HRS.validateValuesLength throws IOE, gets caught in the retries
+   HBASE-892   Cell iteration is broken (Doğacan Güney via Jim Kellerman)
 
   IMPROVEMENTS
    HBASE-890   Alter table operation and also related changes in REST interface

Modified: hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/io/Cell.java
URL: http://svn.apache.org/viewvc/hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/io/Cell.java?rev=698016&r1=698015&r2=698016&view=diff
==============================================================================
--- hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/io/Cell.java (original)
+++ hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/io/Cell.java Mon Sep 22 15:00:31 2008
@@ -161,7 +161,7 @@
     return new CellIterator();
   }
   private class CellIterator implements Iterator<Cell> {
-    private int currentValue = -1;
+    private int currentValue = 0;
     CellIterator() {
     }
     
@@ -170,8 +170,9 @@
     }
     
     public Cell next() {
-      currentValue += 1;
-      return new Cell(values[currentValue], timestamps[currentValue]);
+      Cell c = new Cell(values[currentValue], timestamps[currentValue]);
+      currentValue++;
+      return c;
     }
     
     public void remove() throws UnsupportedOperationException {