You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by st...@apache.org on 2009/10/30 20:29:12 UTC

svn commit: r831430 - in /hadoop/hbase/branches/0.20: CHANGES.txt src/java/org/apache/hadoop/hbase/client/Put.java

Author: stack
Date: Fri Oct 30 19:29:11 2009
New Revision: 831430

URL: http://svn.apache.org/viewvc?rev=831430&view=rev
Log:
HBASE-1941 Put's copy feature has a bug

Modified:
    hadoop/hbase/branches/0.20/CHANGES.txt
    hadoop/hbase/branches/0.20/src/java/org/apache/hadoop/hbase/client/Put.java

Modified: hadoop/hbase/branches/0.20/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/hbase/branches/0.20/CHANGES.txt?rev=831430&r1=831429&r2=831430&view=diff
==============================================================================
--- hadoop/hbase/branches/0.20/CHANGES.txt (original)
+++ hadoop/hbase/branches/0.20/CHANGES.txt Fri Oct 30 19:29:11 2009
@@ -22,6 +22,7 @@
    HBASE-1927  Scanners not closed properly in certain circumstances
    HBASE-1934  NullPointerException in ClientScanner (Andrew Purtell via Stack)
    HBASE-1946  Unhandled exception at regionserver (Dmitriy Lyfar via Stack)
+   HBASE-1941  Put's copy feature has a bug (Dave Latham via Stack) 
 
   IMPROVEMENTS
    HBASE-1899  Use scanner caching in shell count

Modified: hadoop/hbase/branches/0.20/src/java/org/apache/hadoop/hbase/client/Put.java
URL: http://svn.apache.org/viewvc/hadoop/hbase/branches/0.20/src/java/org/apache/hadoop/hbase/client/Put.java?rev=831430&r1=831429&r2=831430&view=diff
==============================================================================
--- hadoop/hbase/branches/0.20/src/java/org/apache/hadoop/hbase/client/Put.java (original)
+++ hadoop/hbase/branches/0.20/src/java/org/apache/hadoop/hbase/client/Put.java Fri Oct 30 19:29:11 2009
@@ -90,6 +90,8 @@
    */
   public Put(Put putToCopy) {
     this(putToCopy.getRow(), putToCopy.getRowLock());
+    this.timestamp = putToCopy.timestamp;
+    this.writeToWAL = putToCopy.writeToWAL;
     this.familyMap = 
       new TreeMap<byte [], List<KeyValue>>(Bytes.BYTES_COMPARATOR);
     for(Map.Entry<byte [], List<KeyValue>> entry :
@@ -217,6 +219,8 @@
   
   /**
    * Method for setting the timestamp
+   * NOTE - This does not affect the timestamp for values previously added to this Put.
+   * It only affects the timestamp for values added after this method is called.
    * @param timestamp
    */
   public Put setTimeStamp(long timestamp) {