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 {