----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18065/ -----------------------------------------------------------
(Updated March 7, 2014, 10:21 a.m.) Review request for hive and Ashutosh Chauhan. Changes ------- Addressed unit test failures. Please check 3 .q.out changes. Those were required because this patch removes one extra copy phase. Bugs: HIVE-6024 https://issues.apache.org/jira/browse/HIVE-6024 Repository: hive-git Description ------- Excerpt from the JIRA: "Load data command creates an additional copy task only when its loading from local It doesn't create this additional copy task while loading from DFS though." Diffs (updated) ----- itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/history/TestHiveHistory.java 8beef09 ql/src/java/org/apache/hadoop/hive/ql/exec/MoveTask.java a190155 ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java 05a2da7 ql/src/java/org/apache/hadoop/hive/ql/metadata/Table.java 8318be1 ql/src/java/org/apache/hadoop/hive/ql/parse/LoadSemanticAnalyzer.java 9ec22ca ql/src/java/org/apache/hadoop/hive/ql/plan/MoveWork.java 407450e ql/src/test/org/apache/hadoop/hive/ql/exec/TestExecDriver.java 5991aae ql/src/test/queries/clientpositive/load_local_dir_test.q PRE-CREATION ql/src/test/results/clientpositive/input4.q.out 9b169f9 ql/src/test/results/clientpositive/load_local_dir_test.q.out PRE-CREATION ql/src/test/results/clientpositive/stats11.q.out ce1197e ql/src/test/results/clientpositive/stats3.q.out a14e449 Diff: https://reviews.apache.org/r/18065/diff/ Testing ------- Ran some existing q tests with "LOAD DATA LOCAL INPATH". Thanks, Mohammad Islam