-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54042/
-----------------------------------------------------------
(Updated Nov. 28, 2016, 9:52 p.m.)
Review request for hive.
Bugs: HIVE-15114
https://issues.apache.org/jira/browse/HIVE-15114
Repository: hive-git
Description
-------
This patch removes an extra MoveTask from the CONDITIONAL TASK added to the
query plan. It would be used only as a Blobstorage optimization.
In HDFS, this two tasks are just simple atomic renames, but on Blobstorage, a
rename is too expensive.
Diffs (updated)
-----
itests/hive-blobstore/src/test/queries/clientpositive/conditional_task_optimization.q
PRE-CREATION
itests/hive-blobstore/src/test/results/clientpositive/conditional_task_optimization.q.out
PRE-CREATION
itests/hive-blobstore/src/test/results/clientpositive/insert_into.q.out
00ad136ae40a5ed2689ad8ed2f882bbcd238324d
ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java
88d5afadd690c3dce276b72a37608d30ddeea271
ql/src/test/org/apache/hadoop/hive/ql/optimizer/TestGenMapRedUtilsCreateConditionalTask.java
PRE-CREATION
Diff: https://reviews.apache.org/r/54042/diff/
Testing
-------
# Testa added
mvn test -Dtest=TestBlobstoreCliDriver -Dqfile=conditional_task_optimization.q
# All tests
mvn test -Dtest=TestBlobstoreCliDriver
mvn test -Dtest=TestBlobstoreNegativeCliDriver
Thanks,
Sergio Pena
