-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26001/
-----------------------------------------------------------

(Updated Sept. 24, 2014, 9:04 p.m.)


Review request for hive, Brock Noland and Xuefu Zhang.


Changes
-------

Made these qfiles as spark-only tests.


Bugs: hive-8233
    https://issues.apache.org/jira/browse/hive-8233


Repository: hive-git


Description
-------

Right now, for multi-table insertion, we will start from multiple 
FileSinkOperators, and break from their lowest common ancestor, adding 
temporary FileSinkOperator and TableScanOperators. A special case is when the 
LCA is a ForwardOperator, in which case we don't break it, since it's already 
been optimized.
However, there's a issue, considering the following plan:
      ...
      RS_0
       |
      FOR
       |
     /   \
   GBY_1  GBY_2
    |     |
   ...   ...
    |     |
   RS_1  RS_2
    |     |
   ...   ...
    |     |
   FS_1  FS_2
which may result to:
          RW
         /  \
       RW    RW
Hence, because of the issue in HIVE-7731 and HIVE-8118, both downstream 
branches will get duplicated (and same) input.


Diffs (updated)
-----

  itests/src/test/resources/testconfiguration.properties 637fbc1 
  
ql/src/test/queries/clientpositive/spark_groupby7_noskew_multi_single_reducer.q 
PRE-CREATION 
  ql/src/test/queries/clientpositive/spark_groupby8.q PRE-CREATION 
  ql/src/test/queries/clientpositive/spark_groupby8_map.q PRE-CREATION 
  ql/src/test/queries/clientpositive/spark_groupby8_map_skew.q PRE-CREATION 
  ql/src/test/queries/clientpositive/spark_groupby8_noskew.q PRE-CREATION 
  ql/src/test/queries/clientpositive/spark_groupby9.q PRE-CREATION 
  
ql/src/test/queries/clientpositive/spark_groupby_multi_insert_common_distinct.q 
PRE-CREATION 
  ql/src/test/queries/clientpositive/spark_union17.q PRE-CREATION 
  
ql/src/test/results/clientpositive/spark/spark_groupby7_noskew_multi_single_reducer.q.out
 PRE-CREATION 
  ql/src/test/results/clientpositive/spark/spark_groupby8.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/spark/spark_groupby8_map.q.out 
PRE-CREATION 
  ql/src/test/results/clientpositive/spark/spark_groupby8_map_skew.q.out 
PRE-CREATION 
  ql/src/test/results/clientpositive/spark/spark_groupby8_noskew.q.out 
PRE-CREATION 
  ql/src/test/results/clientpositive/spark/spark_groupby9.q.out PRE-CREATION 
  
ql/src/test/results/clientpositive/spark/spark_groupby_multi_insert_common_distinct.q.out
 PRE-CREATION 
  ql/src/test/results/clientpositive/spark/spark_union17.q.out PRE-CREATION 

Diff: https://reviews.apache.org/r/26001/diff/


Testing
-------


Thanks,

Chao Sun

Reply via email to