hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Yin Huai" <h...@cse.ohio-state.edu>
Subject Re: Review Request 12795: [HIVE-4827] Merge a Map-only job to its following MapReduce job with multiple inputs
Date Sat, 27 Jul 2013 00:21:16 GMT

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

(Updated July 27, 2013, 12:21 a.m.)


Review request for hive.


Changes
-------

minor update. Changed the method name from CommonJoinTaskDispatcher.mergeMapJoinTaskWithMapReduceTask
to CommonJoinTaskDispatcher.mergeMapJoinTaskIntoItsChildMapRedTask


Bugs: HIVE-4827
    https://issues.apache.org/jira/browse/HIVE-4827


Repository: hive-git


Description
-------

https://issues.apache.org/jira/browse/HIVE-4827


Diffs (updated)
-----

  common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 8f1af64 
  conf/hive-default.xml.template 69b85dc 
  ql/src/java/org/apache/hadoop/hive/ql/exec/OperatorUtils.java 66b84ff 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java b5a9291 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/CommonJoinTaskDispatcher.java 5340d3c

  ql/src/test/queries/clientpositive/auto_join33.q 5c85842 
  ql/src/test/queries/clientpositive/correlationoptimizer1.q 2adf855 
  ql/src/test/queries/clientpositive/correlationoptimizer3.q fcbb764 
  ql/src/test/queries/clientpositive/correlationoptimizer4.q 0e84cb7 
  ql/src/test/queries/clientpositive/correlationoptimizer5.q 1900f5d 
  ql/src/test/queries/clientpositive/correlationoptimizer6.q 88d790c 
  ql/src/test/queries/clientpositive/correlationoptimizer7.q 9b18972 
  ql/src/test/queries/clientpositive/multiMapJoin1.q 86b0586 
  ql/src/test/queries/clientpositive/multiMapJoin2.q PRE-CREATION 
  ql/src/test/queries/clientpositive/union34.q a88e395 
  ql/src/test/results/clientpositive/auto_join0.q.out 652cb76 
  ql/src/test/results/clientpositive/auto_join10.q.out deb8eb5 
  ql/src/test/results/clientpositive/auto_join11.q.out 939f512 
  ql/src/test/results/clientpositive/auto_join12.q.out 23ed0fc 
  ql/src/test/results/clientpositive/auto_join13.q.out 7e0f41d 
  ql/src/test/results/clientpositive/auto_join15.q.out aa40cff 
  ql/src/test/results/clientpositive/auto_join16.q.out e8f1435 
  ql/src/test/results/clientpositive/auto_join2.q.out a11f347 
  ql/src/test/results/clientpositive/auto_join20.q.out 13722ec 
  ql/src/test/results/clientpositive/auto_join21.q.out 79693fe 
  ql/src/test/results/clientpositive/auto_join22.q.out 6f418db 
  ql/src/test/results/clientpositive/auto_join23.q.out 2755ee1 
  ql/src/test/results/clientpositive/auto_join24.q.out c7e872e 
  ql/src/test/results/clientpositive/auto_join26.q.out 7268755 
  ql/src/test/results/clientpositive/auto_join28.q.out 407303c 
  ql/src/test/results/clientpositive/auto_join29.q.out dec2187 
  ql/src/test/results/clientpositive/auto_join32.q.out 312664a 
  ql/src/test/results/clientpositive/auto_join33.q.out 8fc0e84 
  ql/src/test/results/clientpositive/auto_sortmerge_join_10.q.out da375f6 
  ql/src/test/results/clientpositive/auto_sortmerge_join_11.q.out 42e25fa 
  ql/src/test/results/clientpositive/auto_sortmerge_join_12.q.out 2ec3cf3 
  ql/src/test/results/clientpositive/auto_sortmerge_join_9.q.out 6add99a 
  ql/src/test/results/clientpositive/correlationoptimizer1.q.out db3bd78 
  ql/src/test/results/clientpositive/correlationoptimizer3.q.out cfa7eff 
  ql/src/test/results/clientpositive/correlationoptimizer4.q.out 285a54f 
  ql/src/test/results/clientpositive/correlationoptimizer6.q.out b0438e6 
  ql/src/test/results/clientpositive/correlationoptimizer7.q.out f8db2bf 
  ql/src/test/results/clientpositive/join28.q.out 60165e2 
  ql/src/test/results/clientpositive/join32.q.out af37f54 
  ql/src/test/results/clientpositive/join33.q.out af37f54 
  ql/src/test/results/clientpositive/join_star.q.out 797b892 
  ql/src/test/results/clientpositive/mapjoin_filter_on_outerjoin.q.out ca21c6c 
  ql/src/test/results/clientpositive/mapjoin_mapjoin.q.out 2f5f613 
  ql/src/test/results/clientpositive/mapjoin_subquery.q.out 8243c2c 
  ql/src/test/results/clientpositive/mapjoin_subquery2.q.out 292abe4 
  ql/src/test/results/clientpositive/mapjoin_test_outer.q.out 37817d9 
  ql/src/test/results/clientpositive/multiMapJoin1.q.out a3f5c53 
  ql/src/test/results/clientpositive/multiMapJoin2.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/multi_join_union.q.out 5182bdf 
  ql/src/test/results/clientpositive/union34.q.out 166062a 

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


Testing
-------

Running tests.


Thanks,

Yin Huai


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message