You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by st...@apache.org on 2007/12/11 18:17:31 UTC

svn commit: r603307 - /lucene/hadoop/trunk/src/contrib/hbase/src/java/org/apache/hadoop/hbase/HTable.java

Author: stack
Date: Tue Dec 11 09:17:29 2007
New Revision: 603307

URL: http://svn.apache.org/viewvc?rev=603307&view=rev
Log:
HADOOP-2243 getRow returns empty Map if no-such row.. should return null
Intentionally not added to the CHANGES.txt because way hbase currently works
is correct; this commit just adds a bit of clarifying javadoc.

Modified:
    lucene/hadoop/trunk/src/contrib/hbase/src/java/org/apache/hadoop/hbase/HTable.java

Modified: lucene/hadoop/trunk/src/contrib/hbase/src/java/org/apache/hadoop/hbase/HTable.java
URL: http://svn.apache.org/viewvc/lucene/hadoop/trunk/src/contrib/hbase/src/java/org/apache/hadoop/hbase/HTable.java?rev=603307&r1=603306&r2=603307&view=diff
==============================================================================
--- lucene/hadoop/trunk/src/contrib/hbase/src/java/org/apache/hadoop/hbase/HTable.java (original)
+++ lucene/hadoop/trunk/src/contrib/hbase/src/java/org/apache/hadoop/hbase/HTable.java Tue Dec 11 09:17:29 2007
@@ -352,7 +352,7 @@
    * Get all the data for the specified row at the latest timestamp
    * 
    * @param row row key
-   * @return map of colums to values
+   * @return Map of columns to values.  Map is empty if row does not exist.
    * @throws IOException
    */
   public SortedMap<Text, byte[]> getRow(Text row) throws IOException {
@@ -364,7 +364,7 @@
    * 
    * @param row row key
    * @param ts timestamp
-   * @return map of colums to values
+   * @return Map of columns to values.  Map is empty if row does not exist.
    * @throws IOException
    */
   public SortedMap<Text, byte[]> getRow(Text row, long ts) throws IOException {
@@ -650,12 +650,12 @@
   }
   
   /** 
-   * Change a value for the specified column.
+   * Update a value for the specified column.
    * Runs {@link #abort(long)} if exception thrown.
    *
    * @param lockid lock id returned from startUpdate
    * @param column column whose value is being set
-   * @param val new value for column
+   * @param val new value for column.  Cannot be null.
    */
   public void put(long lockid, Text column, byte val[]) {
     checkClosed();
@@ -667,12 +667,12 @@
   }
   
   /** 
-   * Change a value for the specified column.
+   * Update a value for the specified column.
    * Runs {@link #abort(long)} if exception thrown.
    *
    * @param lockid lock id returned from startUpdate
    * @param column column whose value is being set
-   * @param val new value for column
+   * @param val new value for column.  Cannot be null.
    * @throws IOException throws this if the writable can't be
    * converted into a byte array 
    */