-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/65130/
-----------------------------------------------------------
(Updated Feb. 2, 2018, 12:24 a.m.)
Review request for hive, Eugene Koifman, Gopal V, Jason Dere, and Thejas Nair.
Changes
-------
Implemented review comments by Jason.
Made the two fields added to Table struct optional and added them at the end.
Bugs: HIVE-18350
https://issues.apache.org/jira/browse/HIVE-18350
Repository: hive-git
Description
-------
Made changes for both bucketed and non-bucketed tables.
Added a positive test for non-bucketed table which renames the loaded file.
Added couple of negative tests for bucketed table which reject a load with
inconsistent file name.
Diffs (updated)
-----
ql/src/java/org/apache/hadoop/hive/ql/exec/tez/CustomPartitionVertex.java
26afe90faa
ql/src/java/org/apache/hadoop/hive/ql/exec/tez/CustomVertexConfiguration.java
ef5e7edcd6
ql/src/java/org/apache/hadoop/hive/ql/exec/tez/DagUtils.java 9885038588
ql/src/java/org/apache/hadoop/hive/ql/metadata/Table.java 9b0ffe0e91
ql/src/java/org/apache/hadoop/hive/ql/optimizer/ConvertJoinMapJoin.java
dc698c8de8
ql/src/java/org/apache/hadoop/hive/ql/optimizer/metainfo/annotation/OpTraitsRulesProcFactory.java
69d9f3125a
ql/src/java/org/apache/hadoop/hive/ql/optimizer/spark/SparkMapJoinOptimizer.java
bacc44482a
ql/src/java/org/apache/hadoop/hive/ql/parse/LoadSemanticAnalyzer.java
5868d4dd56
ql/src/java/org/apache/hadoop/hive/ql/plan/OpTraits.java 9621c3be53
ql/src/test/queries/clientpositive/auto_sortmerge_join_2.q e5fdcb57e4
ql/src/test/queries/clientpositive/auto_sortmerge_join_4.q abf09e5534
ql/src/test/queries/clientpositive/auto_sortmerge_join_5.q b85c4a7aa3
ql/src/test/queries/clientpositive/auto_sortmerge_join_7.q bd780861e3
ql/src/test/results/clientnegative/bucket_mapjoin_mismatch1.q.out b9c2e6f827
ql/src/test/results/clientpositive/auto_sortmerge_join_2.q.out 5cfc35aa73
ql/src/test/results/clientpositive/auto_sortmerge_join_4.q.out 0d586fd26b
ql/src/test/results/clientpositive/auto_sortmerge_join_5.q.out 45704d1253
ql/src/test/results/clientpositive/auto_sortmerge_join_7.q.out 1959075912
ql/src/test/results/clientpositive/llap/auto_sortmerge_join_2.q.out
054b0d00be
ql/src/test/results/clientpositive/llap/auto_sortmerge_join_4.q.out
95d329862c
ql/src/test/results/clientpositive/llap/auto_sortmerge_join_5.q.out
e711715aa5
ql/src/test/results/clientpositive/llap/auto_sortmerge_join_7.q.out
53c685cb11
ql/src/test/results/clientpositive/spark/auto_sortmerge_join_2.q.out
8cfa113794
ql/src/test/results/clientpositive/spark/auto_sortmerge_join_4.q.out
fce5e0cfc4
ql/src/test/results/clientpositive/spark/auto_sortmerge_join_5.q.out
8250eca099
ql/src/test/results/clientpositive/spark/auto_sortmerge_join_7.q.out
eb813c1734
standalone-metastore/src/gen/thrift/gen-cpp/hive_metastore_types.h df646a7d17
standalone-metastore/src/gen/thrift/gen-cpp/hive_metastore_types.cpp
27f8c0f2fc
standalone-metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/Table.java
f317b0393f
standalone-metastore/src/gen/thrift/gen-php/metastore/Types.php 6878ee1be7
standalone-metastore/src/gen/thrift/gen-py/hive_metastore/ttypes.py
25e9a889b2
standalone-metastore/src/gen/thrift/gen-rb/hive_metastore_types.rb 3a11a0582a
standalone-metastore/src/main/thrift/hive_metastore.thrift 93f3e53de2
Diff: https://reviews.apache.org/r/65130/diff/7/
Changes: https://reviews.apache.org/r/65130/diff/6-7/
Testing
-------
Thanks,
Deepak Jaiswal
