You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@drill.apache.org by jn...@apache.org on 2015/04/22 08:10:15 UTC
[12/13] drill git commit: DRILL-2761: ParquetGroupScan's copy
constructor should copy the content of a collection data member.
DRILL-2761: ParquetGroupScan's copy constructor should copy the content of a collection data member.
Project: http://git-wip-us.apache.org/repos/asf/drill/repo
Commit: http://git-wip-us.apache.org/repos/asf/drill/commit/e462d14e
Tree: http://git-wip-us.apache.org/repos/asf/drill/tree/e462d14e
Diff: http://git-wip-us.apache.org/repos/asf/drill/diff/e462d14e
Branch: refs/heads/master
Commit: e462d14e63e4b396935f611cba5183c6f5d62a8f
Parents: e8f60b2
Author: Jinfeng Ni <jn...@apache.org>
Authored: Fri Apr 10 15:19:38 2015 -0700
Committer: Jinfeng Ni <jn...@apache.org>
Committed: Tue Apr 21 16:32:57 2015 -0700
----------------------------------------------------------------------
.../drill/exec/store/parquet/ParquetGroupScan.java | 11 +++++------
1 file changed, 5 insertions(+), 6 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/drill/blob/e462d14e/exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/ParquetGroupScan.java
----------------------------------------------------------------------
diff --git a/exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/ParquetGroupScan.java b/exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/ParquetGroupScan.java
index 21b9b48..64c245d 100644
--- a/exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/ParquetGroupScan.java
+++ b/exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/ParquetGroupScan.java
@@ -156,16 +156,15 @@ public class ParquetGroupScan extends AbstractFileGroupScan {
* This is used to clone another copy of the group scan.
*/
private ParquetGroupScan(ParquetGroupScan that) {
- super(that);
- this.columns = that.columns;
- this.endpointAffinities = that.endpointAffinities;
- this.entries = that.entries;
+ this.columns = that.columns == null ? null : Lists.newArrayList(that.columns);
+ this.endpointAffinities = that.endpointAffinities == null ? null : Lists.newArrayList(that.endpointAffinities);
+ this.entries = that.entries == null ? null : Lists.newArrayList(that.entries);
this.formatConfig = that.formatConfig;
this.formatPlugin = that.formatPlugin;
this.fs = that.fs;
- this.mappings = that.mappings;
+ this.mappings = that.mappings == null ? null : ArrayListMultimap.create(that.mappings);
this.rowCount = that.rowCount;
- this.rowGroupInfos = that.rowGroupInfos;
+ this.rowGroupInfos = that.rowGroupInfos == null ? null : Lists.newArrayList(that.rowGroupInfos);
this.selectionRoot = that.selectionRoot;
this.columnValueCounts = that.columnValueCounts;
}