> On Jan. 27, 2014, 9:52 p.m., Vikram Dixit Kumaraswamy wrote:
> > itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java, line 90
> > <https://reviews.apache.org/r/16728/diff/4/?file=431458#file431458line90>
> >
> >     This would break tez tests.

Included by mistake (described in HIVE-6241). I'll remove this.


> On Jan. 27, 2014, 9:52 p.m., Vikram Dixit Kumaraswamy wrote:
> > itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java, line 369
> > <https://reviews.apache.org/r/16728/diff/4/?file=431458#file431458line369>
> >
> >     This would eliminate tez unit tests. Was this intentional?

Same with above one


> On Jan. 27, 2014, 9:52 p.m., Vikram Dixit Kumaraswamy wrote:
> > ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/LocalMapJoinProcFactory.java,
> >  line 142
> > <https://reviews.apache.org/r/16728/diff/4/?file=431471#file431471line142>
> >
> >     Could you raise a jira for this.

Ok. sure.


> On Jan. 27, 2014, 9:52 p.m., Vikram Dixit Kumaraswamy wrote:
> > ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/LocalMapJoinProcFactory.java,
> >  line 165
> > <https://reviews.apache.org/r/16728/diff/4/?file=431471#file431471line165>
> >
> >     can it be only these 2 operators? Maybe common join operator can be 
> > used?

Logically, it might possible but in real, it seemed not possible to have 
CommonJoinOperator as a parent which would be in reducer task. Tez could 
exploit that but I don't know well on tez.


- Navis


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


On Jan. 28, 2014, 1:37 a.m., Navis Ryu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/16728/
> -----------------------------------------------------------
> 
> (Updated Jan. 28, 2014, 1:37 a.m.)
> 
> 
> Review request for hive.
> 
> 
> Bugs: HIVE-6144
>     https://issues.apache.org/jira/browse/HIVE-6144
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> For map join, all data in small aliases are hashed and stored into temporary 
> file in MapRedLocalTask. But for some aliases without filter or projection, 
> it seemed not necessary to do that. For example.
> 
> {noformat}
> select a.* from src a join src b on a.key=b.key;
> {noformat}
> 
> makes plan like this.
> {noformat}
> STAGE PLANS:
>   Stage: Stage-4
>     Map Reduce Local Work
>       Alias -> Map Local Tables:
>         a 
>           Fetch Operator
>             limit: -1
>       Alias -> Map Local Operator Tree:
>         a 
>           TableScan
>             alias: a
>             HashTable Sink Operator
>               condition expressions:
>                 0 {key} {value}
>                 1 
>               handleSkewJoin: false
>               keys:
>                 0 [Column[key]]
>                 1 [Column[key]]
>               Position of Big Table: 1
> 
>   Stage: Stage-3
>     Map Reduce
>       Alias -> Map Operator Tree:
>         b 
>           TableScan
>             alias: b
>             Map Join Operator
>               condition map:
>                    Inner Join 0 to 1
>               condition expressions:
>                 0 {key} {value}
>                 1 
>               handleSkewJoin: false
>               keys:
>                 0 [Column[key]]
>                 1 [Column[key]]
>               outputColumnNames: _col0, _col1
>               Position of Big Table: 1
>               Select Operator
>                 File Output Operator
>       Local Work:
>         Map Reduce Local Work
>   Stage: Stage-0
>     Fetch Operator
> {noformat}
> 
> table src(a) is fetched and stored as-is in MRLocalTask. With this patch, 
> plan can be like below.
> {noformat}
>   Stage: Stage-3
>     Map Reduce
>       Alias -> Map Operator Tree:
>         b 
>           TableScan
>             alias: b
>             Map Join Operator
>               condition map:
>                    Inner Join 0 to 1
>               condition expressions:
>                 0 {key} {value}
>                 1 
>               handleSkewJoin: false
>               keys:
>                 0 [Column[key]]
>                 1 [Column[key]]
>               outputColumnNames: _col0, _col1
>               Position of Big Table: 1
>               Select Operator
>                   File Output Operator
>       Local Work:
>         Map Reduce Local Work
>           Alias -> Map Local Tables:
>             a 
>               Fetch Operator
>                 limit: -1
>           Alias -> Map Local Operator Tree:
>             a 
>               TableScan
>                 alias: a
>           Has Any Stage Alias: false
>   Stage: Stage-0
>     Fetch Operator
> {noformat}
> 
> 
> Diffs
> -----
> 
>   common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 84ee78f 
>   conf/hive-default.xml.template 66d22f9 
>   
> itests/util/src/main/java/org/apache/hadoop/hive/ql/hooks/MapJoinCounterHook.java
>  1b0d57e 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/AbstractMapJoinOperator.java 
> d8f4eb4 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/HashTableLoader.java a080fcc 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/HashTableSinkOperator.java 
> fc08b28 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/JoinUtil.java 1e0314d 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/MapJoinOperator.java bdc85b9 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/Task.java 56676df 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/TemporaryHashSinkOperator.java 
> PRE-CREATION 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/mr/ExecDriver.java 22e5777 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/mr/HashTableLoader.java 58484af 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/mr/MapredLocalTask.java 2d2508d 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/tez/HashTableLoader.java 2df8ab9 
>   
> ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/LocalMapJoinProcFactory.java
>  5a53e15 
>   
> ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/MapJoinResolver.java 
> 83b8d6e 
>   
> ql/src/java/org/apache/hadoop/hive/ql/plan/ConditionalResolverCommonJoin.java 
> ebccb14 
>   ql/src/java/org/apache/hadoop/hive/ql/plan/HashTableSinkDesc.java c30da56 
>   ql/src/java/org/apache/hadoop/hive/ql/plan/MapredLocalWork.java 709c50e 
>   ql/src/test/queries/clientpositive/auto_join_without_localtask.q 
> PRE-CREATION 
>   ql/src/test/results/clientnegative/bucket_mapjoin_mismatch1.q.out 0595cd6 
>   ql/src/test/results/clientnegative/file_with_header_footer_negative.q.out 
> fa261b3 
>   ql/src/test/results/clientnegative/sortmerge_mapjoin_mismatch_1.q.out 
> bca069a 
>   ql/src/test/results/clientpositive/auto_join1.q.out b93c10f 
>   ql/src/test/results/clientpositive/auto_join15.q.out 6bfcfc7 
>   ql/src/test/results/clientpositive/auto_join17.q.out 698270e 
>   ql/src/test/results/clientpositive/auto_join19.q.out 247b081 
>   ql/src/test/results/clientpositive/auto_join2.q.out 756cfa1 
>   ql/src/test/results/clientpositive/auto_join21.q.out 053475f 
>   ql/src/test/results/clientpositive/auto_join22.q.out 6e5ae4c 
>   ql/src/test/results/clientpositive/auto_join24.q.out dbecb2c 
>   ql/src/test/results/clientpositive/auto_join25.q.out d6d0c90 
>   ql/src/test/results/clientpositive/auto_join26.q.out 7ca6bb9 
>   ql/src/test/results/clientpositive/auto_join28.q.out 488e15c 
>   ql/src/test/results/clientpositive/auto_join29.q.out 7798a0e 
>   ql/src/test/results/clientpositive/auto_join3.q.out 4279e8c 
>   ql/src/test/results/clientpositive/auto_join30.q.out a46ea2a 
>   ql/src/test/results/clientpositive/auto_join31.q.out 4504426 
>   ql/src/test/results/clientpositive/auto_join32.q.out da4ef55 
>   ql/src/test/results/clientpositive/auto_join9.q.out d2589ed 
>   ql/src/test/results/clientpositive/auto_join_without_localtask.q.out 
> PRE-CREATION 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_1.q.out 8e11616 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_11.q.out 95e86e1 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_12.q.out fd8e9cc 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_14.q.out 3c20e0f 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_15.q.out 6189df6 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_2.q.out c00b519 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_3.q.out 42bab30 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_4.q.out ec33acd 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_5.q.out f8c164a 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_6.q.out 6ee49a7 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_7.q.out 3f2672e 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_8.q.out 4cfd022 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_9.q.out 7b9f811 
>   ql/src/test/results/clientpositive/bucket_map_join_1.q.out f052025 
>   ql/src/test/results/clientpositive/bucket_map_join_2.q.out eb51e23 
>   ql/src/test/results/clientpositive/bucketcontext_1.q.out 9bdc1ee 
>   ql/src/test/results/clientpositive/bucketcontext_2.q.out d5d67dc 
>   ql/src/test/results/clientpositive/bucketcontext_3.q.out b226246 
>   ql/src/test/results/clientpositive/bucketcontext_4.q.out c27d927 
>   ql/src/test/results/clientpositive/bucketcontext_5.q.out 819362f 
>   ql/src/test/results/clientpositive/bucketcontext_6.q.out d7b1ad7 
>   ql/src/test/results/clientpositive/bucketcontext_7.q.out 80749f0 
>   ql/src/test/results/clientpositive/bucketcontext_8.q.out b8a4d4f 
>   ql/src/test/results/clientpositive/bucketmapjoin1.q.out 8d24d6f 
>   ql/src/test/results/clientpositive/bucketmapjoin10.q.out 371af7d 
>   ql/src/test/results/clientpositive/bucketmapjoin11.q.out fcfa41f 
>   ql/src/test/results/clientpositive/bucketmapjoin12.q.out aa50cfe 
>   ql/src/test/results/clientpositive/bucketmapjoin13.q.out eceeade 
>   ql/src/test/results/clientpositive/bucketmapjoin2.q.out 1dfe3e7 
>   ql/src/test/results/clientpositive/bucketmapjoin3.q.out ba1962d 
>   ql/src/test/results/clientpositive/bucketmapjoin4.q.out a9c4bce 
>   ql/src/test/results/clientpositive/bucketmapjoin5.q.out 4812a21 
>   ql/src/test/results/clientpositive/bucketmapjoin7.q.out f8ea2e2 
>   ql/src/test/results/clientpositive/bucketmapjoin8.q.out 8e2be73 
>   ql/src/test/results/clientpositive/bucketmapjoin9.q.out 38b9c87 
>   ql/src/test/results/clientpositive/bucketmapjoin_negative.q.out 0139546 
>   ql/src/test/results/clientpositive/bucketmapjoin_negative2.q.out bf00114 
>   ql/src/test/results/clientpositive/bucketmapjoin_negative3.q.out 2cc1a0f 
>   ql/src/test/results/clientpositive/bucketsortoptimize_insert_2.q.out 
> 0014891 
>   ql/src/test/results/clientpositive/bucketsortoptimize_insert_4.q.out 
> 14d0b6a 
>   ql/src/test/results/clientpositive/bucketsortoptimize_insert_5.q.out 
> e83ef59 
>   ql/src/test/results/clientpositive/bucketsortoptimize_insert_6.q.out 
> b5828f0 
>   ql/src/test/results/clientpositive/correlationoptimizer1.q.out 3ef5cf2 
>   ql/src/test/results/clientpositive/correlationoptimizer3.q.out e633287 
>   ql/src/test/results/clientpositive/correlationoptimizer4.q.out c037d3c 
>   ql/src/test/results/clientpositive/correlationoptimizer5.q.out 02ba37a 
>   ql/src/test/results/clientpositive/correlationoptimizer6.q.out 557ee9e 
>   ql/src/test/results/clientpositive/correlationoptimizer7.q.out b1fe60a 
>   ql/src/test/results/clientpositive/explain_rearrange.q.out 47b1afb 
>   ql/src/test/results/clientpositive/infer_bucket_sort_convert_join.q.out 
> effa6c8 
>   ql/src/test/results/clientpositive/join25.q.out 630d862 
>   ql/src/test/results/clientpositive/join26.q.out 9fca2c3 
>   ql/src/test/results/clientpositive/join27.q.out 0ad8bff 
>   ql/src/test/results/clientpositive/join28.q.out f50072c 
>   ql/src/test/results/clientpositive/join29.q.out 0e0bbb4 
>   ql/src/test/results/clientpositive/join30.q.out 924579c 
>   ql/src/test/results/clientpositive/join31.q.out 85b0196 
>   ql/src/test/results/clientpositive/join32.q.out 9e84d84 
>   ql/src/test/results/clientpositive/join32_lessSize.q.out 0f6f8ec 
>   ql/src/test/results/clientpositive/join33.q.out 9e84d84 
>   ql/src/test/results/clientpositive/join34.q.out 1e2aa91 
>   ql/src/test/results/clientpositive/join35.q.out 83f5ba2 
>   ql/src/test/results/clientpositive/join36.q.out 922ddef 
>   ql/src/test/results/clientpositive/join37.q.out db905bd 
>   ql/src/test/results/clientpositive/join_map_ppr.q.out 131e65c 
>   ql/src/test/results/clientpositive/join_reorder4.q.out b14ea08 
>   ql/src/test/results/clientpositive/join_star.q.out 9b1adbf 
>   ql/src/test/results/clientpositive/mapjoin1.q.out e8b4fa8 
>   ql/src/test/results/clientpositive/mapjoin_distinct.q.out c4dc9d2 
>   ql/src/test/results/clientpositive/mapjoin_hook.q.out 8da5168 
>   ql/src/test/results/clientpositive/mapjoin_mapjoin.q.out 2c9f7f6 
>   ql/src/test/results/clientpositive/mapjoin_subquery.q.out e90a50a 
>   ql/src/test/results/clientpositive/mapjoin_subquery2.q.out 220bf322 
>   ql/src/test/results/clientpositive/mapjoin_test_outer.q.out d66e079 
>   ql/src/test/results/clientpositive/multiMapJoin1.q.out 624b47f 
>   ql/src/test/results/clientpositive/multiMapJoin2.q.out 6d0c7d2 
>   ql/src/test/results/clientpositive/multi_join_union.q.out be6e172 
>   ql/src/test/results/clientpositive/reduce_deduplicate_exclude_join.q.out 
> fdde8df 
>   ql/src/test/results/clientpositive/select_transform_hint.q.out d9ca1fc 
>   ql/src/test/results/clientpositive/skewjoin.q.out 2e03b7e 
>   ql/src/test/results/clientpositive/skewjoin_noskew.q.out fe8da1e 
>   ql/src/test/results/clientpositive/smb_mapjoin_13.q.out b7dc7f2 
>   ql/src/test/results/clientpositive/smb_mapjoin_15.q.out 9adb0d0 
>   ql/src/test/results/clientpositive/sort_merge_join_desc_6.q.out 2011bc3 
>   ql/src/test/results/clientpositive/sort_merge_join_desc_7.q.out 3f2f6ab 
>   ql/src/test/results/clientpositive/stats11.q.out 44ddc03 
>   ql/src/test/results/clientpositive/subq_where_serialization.q.out 94840c4 
>   ql/src/test/results/clientpositive/subquery_in_having.q.out b293ed8 
> 
> Diff: https://reviews.apache.org/r/16728/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Navis Ryu
> 
>

Reply via email to