> On Dec. 2, 2016, 8:58 p.m., Ashutosh Chauhan wrote:
> > ql/src/java/org/apache/hadoop/hive/ql/optimizer/ConvertJoinMapJoin.java, 
> > line 101
> > <https://reviews.apache.org/r/54313/diff/1/?file=1574845#file1574845line101>
> >
> >     Was this just an oversight earlier?

It was exactly that; we might end up calling fallback twice and it was causing 
an Exception to be thrown.


- Jesús


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


On Dec. 2, 2016, 5:24 p.m., Jesús Camacho Rodríguez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/54313/
> -----------------------------------------------------------
> 
> (Updated Dec. 2, 2016, 5:24 p.m.)
> 
> 
> Review request for hive and Ashutosh Chauhan.
> 
> 
> Bugs: HIVE-15251
>     https://issues.apache.org/jira/browse/HIVE-15251
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> HIVE-15251
> 
> 
> Diffs
> -----
> 
>   itests/src/test/resources/testconfiguration.properties 
> e4910e4cda49bc3684e7b9ae5aa0607bd16a0a07 
>   ql/src/java/org/apache/hadoop/hive/ql/ErrorMsg.java 
> b62df35cdddc346d0e5d034b10f7b91f9d17d804 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/CommonJoinOperator.java 
> 5512ee27ce04bb36596288d42281b081ba290d7b 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/JoinUtil.java 
> 6cbcab699141ec143053e6ee3e2d60e93eeeb71a 
>   ql/src/java/org/apache/hadoop/hive/ql/optimizer/ConvertJoinMapJoin.java 
> 2b93e01f36f185ba755bacda89b081fe6286a9ab 
>   ql/src/java/org/apache/hadoop/hive/ql/optimizer/MapJoinProcessor.java 
> c6efd5b21dcf2633ec027ced952fe4b04621d9d4 
>   ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/Vectorizer.java 
> 37baaf694c314a1377ded9b337b785c8c5c783e5 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 
> d55db0accf430724b51a8704838964688a5221ec 
>   ql/src/java/org/apache/hadoop/hive/ql/plan/JoinDesc.java 
> 2ca6b8f1c5a2ad8c586e26d1cbf70d6f1680ebf0 
>   ql/src/test/queries/clientpositive/join46.q PRE-CREATION 
>   ql/src/test/queries/clientpositive/mapjoin46.q PRE-CREATION 
>   ql/src/test/queries/clientpositive/smb_mapjoin_46.q PRE-CREATION 
>   ql/src/test/queries/clientpositive/vectorized_join46.q PRE-CREATION 
>   ql/src/test/results/clientpositive/join46.q.out PRE-CREATION 
>   ql/src/test/results/clientpositive/llap/join46.q.out PRE-CREATION 
>   ql/src/test/results/clientpositive/llap/mapjoin46.q.out PRE-CREATION 
>   ql/src/test/results/clientpositive/llap/vectorized_join46.q.out 
> PRE-CREATION 
>   ql/src/test/results/clientpositive/mapjoin46.q.out PRE-CREATION 
>   ql/src/test/results/clientpositive/smb_mapjoin_46.q.out PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/54313/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Jesús Camacho Rodríguez
> 
>

Reply via email to