Return-Path: Delivered-To: apmail-hadoop-hbase-commits-archive@locus.apache.org Received: (qmail 44784 invoked from network); 10 Sep 2008 21:14:12 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 10 Sep 2008 21:14:12 -0000 Received: (qmail 2858 invoked by uid 500); 10 Sep 2008 21:14:10 -0000 Delivered-To: apmail-hadoop-hbase-commits-archive@hadoop.apache.org Received: (qmail 2841 invoked by uid 500); 10 Sep 2008 21:14:10 -0000 Mailing-List: contact hbase-commits-help@hadoop.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: hbase-dev@hadoop.apache.org Delivered-To: mailing list hbase-commits@hadoop.apache.org Received: (qmail 2832 invoked by uid 99); 10 Sep 2008 21:14:10 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 10 Sep 2008 14:14:10 -0700 X-ASF-Spam-Status: No, hits=-2000.0 required=10.0 tests=ALL_TRUSTED X-Spam-Check-By: apache.org Received: from [140.211.11.4] (HELO eris.apache.org) (140.211.11.4) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 10 Sep 2008 21:13:20 +0000 Received: by eris.apache.org (Postfix, from userid 65534) id 439C62388989; Wed, 10 Sep 2008 14:13:22 -0700 (PDT) Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: svn commit: r693987 - in /hadoop/hbase/trunk: CHANGES.txt src/java/org/apache/hadoop/hbase/io/BatchUpdate.java src/test/org/apache/hadoop/hbase/client/TestBatchUpdate.java Date: Wed, 10 Sep 2008 21:13:22 -0000 To: hbase-commits@hadoop.apache.org From: stack@apache.org X-Mailer: svnmailer-1.0.8 Message-Id: <20080910211322.439C62388989@eris.apache.org> X-Virus-Checked: Checked by ClamAV on apache.org Author: stack Date: Wed Sep 10 14:13:21 2008 New Revision: 693987 URL: http://svn.apache.org/viewvc?rev=693987&view=rev Log: HBASE-882 The BatchUpdate class provides, put(col, cell) and delete(col) but no get(). Modified: hadoop/hbase/trunk/CHANGES.txt hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/io/BatchUpdate.java hadoop/hbase/trunk/src/test/org/apache/hadoop/hbase/client/TestBatchUpdate.java Modified: hadoop/hbase/trunk/CHANGES.txt URL: http://svn.apache.org/viewvc/hadoop/hbase/trunk/CHANGES.txt?rev=693987&r1=693986&r2=693987&view=diff ============================================================================== --- hadoop/hbase/trunk/CHANGES.txt (original) +++ hadoop/hbase/trunk/CHANGES.txt Wed Sep 10 14:13:21 2008 @@ -64,6 +64,8 @@ splits (Jonathan Gray via Stack) HBASE-877 HCM is unable to find table with multiple regions which contains binary (Jonathan Gray via Stack) + HBASE-822 The BatchUpdate class provides, put(col, cell) and delete(col) + but no get() (Ryan Smith via Stack) IMPROVEMENTS HBASE-801 When a table haven't disable, shell could response in a "user Modified: hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/io/BatchUpdate.java URL: http://svn.apache.org/viewvc/hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/io/BatchUpdate.java?rev=693987&r1=693986&r2=693987&view=diff ============================================================================== --- hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/io/BatchUpdate.java (original) +++ hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/io/BatchUpdate.java Wed Sep 10 14:13:21 2008 @@ -23,6 +23,7 @@ import java.io.DataOutput; import java.io.IOException; import java.util.ArrayList; +import java.util.Arrays; import java.util.Iterator; import org.apache.hadoop.hbase.HConstants; @@ -116,6 +117,31 @@ public void setTimestamp(long timestamp) { this.timestamp = timestamp; } + + /** + * Get the current value of the specified column + * + * @param column column name + * @return byte[] the cell value, returns null if the column does not exist. + */ + public synchronized byte[] get(final String column) { + return get(Bytes.toBytes(column)); + } + + /** + * Get the current value of the specified column + * + * @param column column name + * @return byte[] the cell value, returns null if the column does not exist. + */ + public synchronized byte[] get(final byte[] column) { + for (BatchOperation operation: operations) { + if (Arrays.equals(column, operation.getColumn())) { + return operation.getValue(); + } + } + return null; + } /** * Change a value for the specified column Modified: hadoop/hbase/trunk/src/test/org/apache/hadoop/hbase/client/TestBatchUpdate.java URL: http://svn.apache.org/viewvc/hadoop/hbase/trunk/src/test/org/apache/hadoop/hbase/client/TestBatchUpdate.java?rev=693987&r1=693986&r2=693987&view=diff ============================================================================== --- hadoop/hbase/trunk/src/test/org/apache/hadoop/hbase/client/TestBatchUpdate.java (original) +++ hadoop/hbase/trunk/src/test/org/apache/hadoop/hbase/client/TestBatchUpdate.java Wed Sep 10 14:13:21 2008 @@ -22,6 +22,7 @@ import java.io.IOException; import java.io.UnsupportedEncodingException; import java.util.ArrayList; +import java.util.Arrays; import java.util.Map; import org.apache.hadoop.hbase.HBaseClusterTestCase; @@ -85,6 +86,8 @@ bu = new BatchUpdate("row2"); bu.put(CONTENTS, value); + byte[] getValue = bu.get(CONTENTS); + assertTrue(Arrays.equals(getValue, value)); table.commit(bu); byte [][] columns = { CONTENTS };