-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30965/
-----------------------------------------------------------
(Updated Feb. 26, 2015, 12:34 a.m.)
Review request for drill, Jacques Nadeau, Jinfeng Ni, Steven Phillips, and
Venki Korukanti.
Changes
-------
Addressing review comments after code review with Jacques and Venki
Bugs: DRILL-2209
https://issues.apache.org/jira/browse/DRILL-2209
Repository: drill-git
Description
-------
Insert Project operator to add new column "EXPRHASH" with hash expression for
fields that are used for HashToRandomExchange
Remove Project operator after HashRandomExchange (or Demux) since it will
create problems to fields ordering in HashJoin.
Tight this to MuxExchange - so if MuxExchange is enabled, Project is inserted.
Diffs (updated)
-----
exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/HashToRandomExchangePrel.java
372c75d
exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/PrelUtil.java
1adc54f
exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/visitor/InsertLocalExchangeVisitor.java
PRE-CREATION
Diff: https://reviews.apache.org/r/30965/diff/
Testing
-------
Need to add Unit Tests. tested live, run Functional and TPCH tests
Thanks,
Yuliya Feldman
