Return-Path: X-Original-To: apmail-tajo-commits-archive@minotaur.apache.org Delivered-To: apmail-tajo-commits-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 94D8A17F03 for ; Fri, 10 Oct 2014 04:33:09 +0000 (UTC) Received: (qmail 26178 invoked by uid 500); 10 Oct 2014 04:33:09 -0000 Delivered-To: apmail-tajo-commits-archive@tajo.apache.org Received: (qmail 26119 invoked by uid 500); 10 Oct 2014 04:33:08 -0000 Mailing-List: contact commits-help@tajo.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@tajo.apache.org Delivered-To: mailing list commits@tajo.apache.org Received: (qmail 25519 invoked by uid 99); 10 Oct 2014 04:33:08 -0000 Received: from tyr.zones.apache.org (HELO tyr.zones.apache.org) (140.211.11.114) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 10 Oct 2014 04:33:08 +0000 Received: by tyr.zones.apache.org (Postfix, from userid 65534) id 849701925A; Fri, 10 Oct 2014 04:33:08 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: jihoonson@apache.org To: commits@tajo.apache.org Date: Fri, 10 Oct 2014 04:33:41 -0000 Message-Id: In-Reply-To: References: X-Mailer: ASF-Git Admin Mailer Subject: [35/43] git commit: TAJO-1021: Remove the member variable Builder from all classes inherited from ProtoObject. TAJO-1021: Remove the member variable Builder from all classes inherited from ProtoObject. Closes #188 Project: http://git-wip-us.apache.org/repos/asf/tajo/repo Commit: http://git-wip-us.apache.org/repos/asf/tajo/commit/d94727f8 Tree: http://git-wip-us.apache.org/repos/asf/tajo/tree/d94727f8 Diff: http://git-wip-us.apache.org/repos/asf/tajo/diff/d94727f8 Branch: refs/heads/index_support Commit: d94727f893bcac2866da4232ee71fe1b435d4759 Parents: 8967d11 Author: Hyunsik Choi Authored: Wed Oct 8 09:26:28 2014 -0700 Committer: Hyunsik Choi Committed: Wed Oct 8 09:26:28 2014 -0700 ---------------------------------------------------------------------- CHANGES | 3 +++ .../java/org/apache/tajo/catalog/AlterTableDesc.java | 14 +++++--------- .../main/java/org/apache/tajo/catalog/IndexDesc.java | 7 +------ .../main/java/org/apache/tajo/catalog/TableDesc.java | 7 +------ .../tajo/catalog/partition/PartitionMethodDesc.java | 8 -------- .../apache/tajo/catalog/statistics/ColumnStats.java | 8 ++------ 6 files changed, 12 insertions(+), 35 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/tajo/blob/d94727f8/CHANGES ---------------------------------------------------------------------- diff --git a/CHANGES b/CHANGES index cc2b8b8..5075602 100644 --- a/CHANGES +++ b/CHANGES @@ -163,6 +163,9 @@ Release 0.9.0 - unreleased BUG FIXES + TAJO-1021: Remove the member variable Builder from all classes inherited + from ProtoObject. (hyunsik) + TAJO-1104: Using asterisk with GROUP BY causes NPE. (hyunsik) TAJO-1099: LogicalPlanner::convertDataType causes NPE in some cases. http://git-wip-us.apache.org/repos/asf/tajo/blob/d94727f8/tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/AlterTableDesc.java ---------------------------------------------------------------------- diff --git a/tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/AlterTableDesc.java b/tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/AlterTableDesc.java index fcaa4c3..69d5be4 100644 --- a/tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/AlterTableDesc.java +++ b/tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/AlterTableDesc.java @@ -25,11 +25,10 @@ import org.apache.tajo.catalog.proto.CatalogProtos; import org.apache.tajo.common.ProtoObject; import org.apache.tajo.json.GsonObject; +import static org.apache.tajo.catalog.proto.CatalogProtos.AlterTableDescProto; -public class AlterTableDesc implements ProtoObject, GsonObject, Cloneable { - - protected CatalogProtos.AlterTableDescProto.Builder builder = null; +public class AlterTableDesc implements ProtoObject, GsonObject, Cloneable { @Expose protected AlterTableType alterTableType; //required @Expose @@ -44,7 +43,6 @@ public class AlterTableDesc implements ProtoObject, Cloneable { - private IndexDescProto.Builder builder; - private String indexName; // required private String databaseName; // required private String tableName; // required @@ -39,7 +37,6 @@ public class IndexDesc implements ProtoObject, Cloneable { private boolean isAscending = false; // optional [default = false] public IndexDesc() { - this.builder = IndexDescProto.newBuilder(); } public IndexDesc(String idxName, String databaseName, String tableName, Column column, @@ -93,9 +90,7 @@ public class IndexDesc implements ProtoObject, Cloneable { @Override public IndexDescProto getProto() { - if (builder == null) { - builder = IndexDescProto.newBuilder(); - } + IndexDescProto.Builder builder = IndexDescProto.newBuilder(); CatalogProtos.TableIdentifierProto.Builder tableIdentifierBuilder = CatalogProtos.TableIdentifierProto.newBuilder(); if (databaseName != null) { http://git-wip-us.apache.org/repos/asf/tajo/blob/d94727f8/tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/TableDesc.java ---------------------------------------------------------------------- diff --git a/tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/TableDesc.java b/tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/TableDesc.java index c148049..64255d5 100644 --- a/tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/TableDesc.java +++ b/tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/TableDesc.java @@ -38,7 +38,6 @@ import org.apache.tajo.util.TUtil; public class TableDesc implements ProtoObject, GsonObject, Cloneable { private final Log LOG = LogFactory.getLog(TableDesc.class); - protected TableDescProto.Builder builder = null; @Expose protected String tableName; // required @Expose protected Schema schema; @Expose protected TableMeta meta; // required @@ -51,7 +50,6 @@ public class TableDesc implements ProtoObject, GsonObject, Clone @Expose protected Boolean external; // optional public TableDesc() { - builder = TableDescProto.newBuilder(); } public TableDesc(String tableName, Schema schema, TableMeta meta, @@ -180,7 +178,6 @@ public class TableDesc implements ProtoObject, GsonObject, Clone public Object clone() throws CloneNotSupportedException { TableDesc desc = (TableDesc) super.clone(); - desc.builder = TableDescProto.newBuilder(); desc.tableName = tableName; desc.schema = (Schema) schema.clone(); desc.meta = (TableMeta) meta.clone(); @@ -202,9 +199,7 @@ public class TableDesc implements ProtoObject, GsonObject, Clone } public TableDescProto getProto() { - if (builder == null) { - builder = TableDescProto.newBuilder(); - } + TableDescProto.Builder builder = TableDescProto.newBuilder(); if (this.tableName != null) { builder.setTableName(this.tableName); http://git-wip-us.apache.org/repos/asf/tajo/blob/d94727f8/tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/partition/PartitionMethodDesc.java ---------------------------------------------------------------------- diff --git a/tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/partition/PartitionMethodDesc.java b/tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/partition/PartitionMethodDesc.java index 48a105c..721a7a0 100644 --- a/tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/partition/PartitionMethodDesc.java +++ b/tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/partition/PartitionMethodDesc.java @@ -36,8 +36,6 @@ import static org.apache.tajo.catalog.proto.CatalogProtos.TableIdentifierProto; * PartitionMethodDesc presents a table description, including partition type, and partition keys. */ public class PartitionMethodDesc implements ProtoObject, Cloneable, GsonObject { - private CatalogProtos.PartitionMethodProto.Builder builder; - @Expose private String databaseName; // required @Expose private String tableName; // required @Expose private PartitionType partitionType; // required @@ -45,7 +43,6 @@ public class PartitionMethodDesc implements ProtoObject, Cloneable, GsonObject { - private CatalogProtos.ColumnStatsProto.Builder builder = CatalogProtos.ColumnStatsProto.newBuilder(); - @Expose private Column column = null; // required @Expose private Long numDistVals = null; // optional @Expose private Long numNulls = null; // optional @@ -132,7 +130,6 @@ public class ColumnStats implements ProtoObject, public Object clone() throws CloneNotSupportedException { ColumnStats stat = (ColumnStats) super.clone(); - stat.builder = CatalogProtos.ColumnStatsProto.newBuilder(); stat.column = this.column; stat.numDistVals = numDistVals; stat.numNulls = numNulls; @@ -154,9 +151,8 @@ public class ColumnStats implements ProtoObject, @Override public CatalogProtos.ColumnStatsProto getProto() { - if (builder == null) { - builder = CatalogProtos.ColumnStatsProto.newBuilder(); - } + CatalogProtos.ColumnStatsProto.Builder builder = CatalogProtos.ColumnStatsProto.newBuilder(); + if (this.column != null) { builder.setColumn(this.column.getProto()); }