Repository: spark
Updated Branches:
  refs/heads/branch-1.4 ff8b44995 -> d28832299


[MINOR] Minor update for document

Two minor doc errors in `BytesToBytesMap` and `UnsafeRow`.

Author: Liang-Chi Hsieh <vii...@gmail.com>

Closes #5906 from viirya/minor_doc and squashes the following commits:

27f9089 [Liang-Chi Hsieh] Minor update for doc.

(cherry picked from commit b83091ae4589feea78b056827bc3b7659d271e41)
Signed-off-by: Sean Owen <so...@cloudera.com>


Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/d2883229
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/d2883229
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/d2883229

Branch: refs/heads/branch-1.4
Commit: d28832299a12bc6f30df765294770a4da3bf2e07
Parents: ff8b449
Author: Liang-Chi Hsieh <vii...@gmail.com>
Authored: Tue May 5 14:44:02 2015 +0100
Committer: Sean Owen <so...@cloudera.com>
Committed: Tue May 5 14:44:10 2015 +0100

----------------------------------------------------------------------
 .../org/apache/spark/sql/catalyst/expressions/UnsafeRow.java     | 2 +-
 .../main/java/org/apache/spark/unsafe/map/BytesToBytesMap.java   | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/spark/blob/d2883229/sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeRow.java
----------------------------------------------------------------------
diff --git 
a/sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeRow.java
 
b/sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeRow.java
index 0a358ed..bb546b3 100644
--- 
a/sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeRow.java
+++ 
b/sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeRow.java
@@ -145,7 +145,7 @@ public final class UnsafeRow implements MutableRow {
 
   private void assertIndexIsValid(int index) {
     assert index >= 0 : "index (" + index + ") should >= 0";
-    assert index < numFields : "index (" + index + ") should <= " + numFields;
+    assert index < numFields : "index (" + index + ") should < " + numFields;
   }
 
   @Override

http://git-wip-us.apache.org/repos/asf/spark/blob/d2883229/unsafe/src/main/java/org/apache/spark/unsafe/map/BytesToBytesMap.java
----------------------------------------------------------------------
diff --git 
a/unsafe/src/main/java/org/apache/spark/unsafe/map/BytesToBytesMap.java 
b/unsafe/src/main/java/org/apache/spark/unsafe/map/BytesToBytesMap.java
index 4e5ebc4..19d6a16 100644
--- a/unsafe/src/main/java/org/apache/spark/unsafe/map/BytesToBytesMap.java
+++ b/unsafe/src/main/java/org/apache/spark/unsafe/map/BytesToBytesMap.java
@@ -351,9 +351,9 @@ public final class BytesToBytesMap {
      * As an example usage, here's the proper way to store a new key:
      * <p>
      * <pre>
-     *   Location loc = map.lookup(keyBaseOffset, keyBaseObject, 
keyLengthInBytes);
+     *   Location loc = map.lookup(keyBaseObject, keyBaseOffset, 
keyLengthInBytes);
      *   if (!loc.isDefined()) {
-     *     loc.putNewKey(keyBaseOffset, keyBaseObject, keyLengthInBytes, ...)
+     *     loc.putNewKey(keyBaseObject, keyBaseOffset, keyLengthInBytes, ...)
      *   }
      * </pre>
      * <p>


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org

Reply via email to