Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id 72135200CE0 for ; Thu, 27 Jul 2017 01:39:20 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id 70ADF169CA4; Wed, 26 Jul 2017 23:39:20 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 96681169C9E for ; Thu, 27 Jul 2017 01:39:19 +0200 (CEST) Received: (qmail 44947 invoked by uid 500); 26 Jul 2017 23:39:18 -0000 Mailing-List: contact commits-help@hbase.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@hbase.apache.org Delivered-To: mailing list commits@hbase.apache.org Received: (qmail 44937 invoked by uid 99); 26 Jul 2017 23:39:18 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 26 Jul 2017 23:39:18 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id A2649E00C5; Wed, 26 Jul 2017 23:39:18 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: busbey@apache.org To: commits@hbase.apache.org Date: Wed, 26 Jul 2017 23:39:18 -0000 Message-Id: X-Mailer: ASF-Git Admin Mailer Subject: [1/3] hbase git commit: HBASE-18402 Thrift2 should support DeleteFamilyVersion - revert due to missing JIRA number archived-at: Wed, 26 Jul 2017 23:39:20 -0000 Repository: hbase Updated Branches: refs/heads/branch-2 164df6bdd -> 10573c6c9 HBASE-18402 Thrift2 should support DeleteFamilyVersion - revert due to missing JIRA number Project: http://git-wip-us.apache.org/repos/asf/hbase/repo Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/b7045d4f Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/b7045d4f Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/b7045d4f Branch: refs/heads/branch-2 Commit: b7045d4fcfc89551dd6a02de65747fbe440f8107 Parents: 164df6b Author: tedyu Authored: Tue Jul 25 17:38:54 2017 -0700 Committer: Sean Busbey Committed: Wed Jul 26 15:49:16 2017 -0700 ---------------------------------------------------------------------- .../hadoop/hbase/thrift2/ThriftUtilities.java | 47 ++++------ .../hbase/thrift2/generated/TDeleteType.java | 8 +- .../apache/hadoop/hbase/thrift2/hbase.thrift | 4 +- .../thrift2/TestThriftHBaseServiceHandler.java | 91 +------------------- 4 files changed, 21 insertions(+), 129 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hbase/blob/b7045d4f/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/ThriftUtilities.java ---------------------------------------------------------------------- diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/ThriftUtilities.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/ThriftUtilities.java index f318f8e..3807bec 100644 --- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/ThriftUtilities.java +++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/ThriftUtilities.java @@ -268,40 +268,27 @@ public class ThriftUtilities { if (in.isSetColumns()) { out = new Delete(in.getRow()); for (TColumn column : in.getColumns()) { - if (in.isSetDeleteType()) { - switch (in.getDeleteType()) { - case DELETE_COLUMN: - if (column.isSetTimestamp()) { - out.addColumn(column.getFamily(), column.getQualifier(), column.getTimestamp()); - } else { - out.addColumn(column.getFamily(), column.getQualifier()); - } - break; - case DELETE_COLUMNS: - if (column.isSetTimestamp()) { + if (column.isSetQualifier()) { + if (column.isSetTimestamp()) { + if (in.isSetDeleteType() && + in.getDeleteType().equals(TDeleteType.DELETE_COLUMNS)) out.addColumns(column.getFamily(), column.getQualifier(), column.getTimestamp()); - } else { + else + out.addColumn(column.getFamily(), column.getQualifier(), column.getTimestamp()); + } else { + if (in.isSetDeleteType() && + in.getDeleteType().equals(TDeleteType.DELETE_COLUMNS)) out.addColumns(column.getFamily(), column.getQualifier()); - } - break; - case DELETE_FAMILY: - if (column.isSetTimestamp()) { - out.addFamily(column.getFamily(), column.getTimestamp()); - } else { - out.addFamily(column.getFamily()); - } - break; - case DELETE_FAMILY_VERSION: - if (column.isSetTimestamp()) { - out.addFamilyVersion(column.getFamily(), column.getTimestamp()); - } else { - throw new IllegalArgumentException( - "Timestamp is required for TDelete with DeleteFamilyVersion type"); - } - break; + else + out.addColumn(column.getFamily(), column.getQualifier()); } + } else { - throw new IllegalArgumentException("DeleteType is required for TDelete"); + if (column.isSetTimestamp()) { + out.addFamily(column.getFamily(), column.getTimestamp()); + } else { + out.addFamily(column.getFamily()); + } } } } else { http://git-wip-us.apache.org/repos/asf/hbase/blob/b7045d4f/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TDeleteType.java ---------------------------------------------------------------------- diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TDeleteType.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TDeleteType.java index 8c9aa39..5d47508 100644 --- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TDeleteType.java +++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift2/generated/TDeleteType.java @@ -18,9 +18,7 @@ import org.apache.thrift.TEnum; */ public enum TDeleteType implements org.apache.thrift.TEnum { DELETE_COLUMN(0), - DELETE_COLUMNS(1), - DELETE_FAMILY(2), - DELETE_FAMILY_VERSION(3); + DELETE_COLUMNS(1); private final int value; @@ -45,10 +43,6 @@ public enum TDeleteType implements org.apache.thrift.TEnum { return DELETE_COLUMN; case 1: return DELETE_COLUMNS; - case 2: - return DELETE_FAMILY; - case 3: - return DELETE_FAMILY_VERSION; default: return null; } http://git-wip-us.apache.org/repos/asf/hbase/blob/b7045d4f/hbase-thrift/src/main/resources/org/apache/hadoop/hbase/thrift2/hbase.thrift ---------------------------------------------------------------------- diff --git a/hbase-thrift/src/main/resources/org/apache/hadoop/hbase/thrift2/hbase.thrift b/hbase-thrift/src/main/resources/org/apache/hadoop/hbase/thrift2/hbase.thrift index 8c5ef59..e2e5b29 100644 --- a/hbase-thrift/src/main/resources/org/apache/hadoop/hbase/thrift2/hbase.thrift +++ b/hbase-thrift/src/main/resources/org/apache/hadoop/hbase/thrift2/hbase.thrift @@ -75,9 +75,7 @@ struct TResult { */ enum TDeleteType { DELETE_COLUMN = 0, - DELETE_COLUMNS = 1, - DELETE_FAMILY = 2, - DELETE_FAMILY_VERSION = 3 + DELETE_COLUMNS = 1 } /** http://git-wip-us.apache.org/repos/asf/hbase/blob/b7045d4f/hbase-thrift/src/test/java/org/apache/hadoop/hbase/thrift2/TestThriftHBaseServiceHandler.java ---------------------------------------------------------------------- diff --git a/hbase-thrift/src/test/java/org/apache/hadoop/hbase/thrift2/TestThriftHBaseServiceHandler.java b/hbase-thrift/src/test/java/org/apache/hadoop/hbase/thrift2/TestThriftHBaseServiceHandler.java index 7a35d29..3f5c388 100644 --- a/hbase-thrift/src/test/java/org/apache/hadoop/hbase/thrift2/TestThriftHBaseServiceHandler.java +++ b/hbase-thrift/src/test/java/org/apache/hadoop/hbase/thrift2/TestThriftHBaseServiceHandler.java @@ -31,11 +31,8 @@ import org.apache.hadoop.hbase.HColumnDescriptor; import org.apache.hadoop.hbase.HTableDescriptor; import org.apache.hadoop.hbase.TableName; import org.apache.hadoop.hbase.client.Admin; -import org.apache.hadoop.hbase.client.Connection; -import org.apache.hadoop.hbase.client.ConnectionFactory; import org.apache.hadoop.hbase.client.Get; import org.apache.hadoop.hbase.client.Put; -import org.apache.hadoop.hbase.client.Result; import org.apache.hadoop.hbase.client.Scan; import org.apache.hadoop.hbase.client.Increment; import org.apache.hadoop.hbase.client.Delete; @@ -78,8 +75,10 @@ import org.junit.Before; import org.junit.BeforeClass; import org.junit.Rule; import org.junit.Test; +import org.junit.Rule; import org.junit.rules.TestName; import org.junit.experimental.categories.Category; +import org.junit.rules.TestName; import java.io.IOException; import java.io.InterruptedIOException; @@ -442,92 +441,6 @@ public class TestThriftHBaseServiceHandler { } @Test - public void testDeleteFamily() throws Exception { - ThriftHBaseServiceHandler handler = createHandler(); - byte[] rowName = "testDeleteFamily".getBytes(); - ByteBuffer table = wrap(tableAname); - - long timestamp1 = System.currentTimeMillis() - 10; - long timestamp2 = System.currentTimeMillis(); - - List columnValues = new ArrayList(); - TColumnValue columnValueA = - new TColumnValue(wrap(familyAname), wrap(qualifierAname), wrap(valueAname)); - columnValueA.setTimestamp(timestamp1); - columnValues.add(columnValueA); - TPut put = new TPut(wrap(rowName), columnValues); - - put.setColumnValues(columnValues); - - handler.put(table, put); - columnValueA.setTimestamp(timestamp2); - handler.put(table, put); - - TGet get = new TGet(wrap(rowName)); - get.setMaxVersions(2); - TResult result = handler.get(table, get); - assertEquals(2, result.getColumnValuesSize()); - - TDelete delete = new TDelete(wrap(rowName)); - List deleteColumns = new ArrayList(); - TColumn deleteColumn = new TColumn(wrap(familyAname)); - deleteColumns.add(deleteColumn); - delete.setColumns(deleteColumns); - delete.setDeleteType(TDeleteType.DELETE_FAMILY); - - handler.deleteSingle(table, delete); - - get = new TGet(wrap(rowName)); - result = handler.get(table, get); - assertArrayEquals(null, result.getRow()); - assertEquals(0, result.getColumnValuesSize()); - } - - @Test - public void testDeleteFamilyVersion() throws Exception { - ThriftHBaseServiceHandler handler = createHandler(); - byte[] rowName = "testDeleteFamilyVersion".getBytes(); - ByteBuffer table = wrap(tableAname); - - long timestamp1 = System.currentTimeMillis() - 10; - long timestamp2 = System.currentTimeMillis(); - - List columnValues = new ArrayList(); - TColumnValue columnValueA = - new TColumnValue(wrap(familyAname), wrap(qualifierAname), wrap(valueAname)); - columnValueA.setTimestamp(timestamp1); - columnValues.add(columnValueA); - TPut put = new TPut(wrap(rowName), columnValues); - - put.setColumnValues(columnValues); - - handler.put(table, put); - columnValueA.setTimestamp(timestamp2); - handler.put(table, put); - - TGet get = new TGet(wrap(rowName)); - get.setMaxVersions(2); - TResult result = handler.get(table, get); - assertEquals(2, result.getColumnValuesSize()); - - TDelete delete = new TDelete(wrap(rowName)); - List deleteColumns = new ArrayList(); - TColumn deleteColumn = new TColumn(wrap(familyAname)); - deleteColumn.setTimestamp(timestamp1); - deleteColumns.add(deleteColumn); - delete.setColumns(deleteColumns); - delete.setDeleteType(TDeleteType.DELETE_FAMILY_VERSION); - - handler.deleteSingle(table, delete); - - get = new TGet(wrap(rowName)); - result = handler.get(table, get); - assertArrayEquals(rowName, result.getRow()); - assertEquals(1, result.getColumnValuesSize()); - assertEquals(timestamp2, result.getColumnValues().get(0).getTimestamp()); - } - - @Test public void testIncrement() throws Exception { ThriftHBaseServiceHandler handler = createHandler(); byte[] rowName = "testIncrement".getBytes();