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

(Updated Feb. 18, 2015, 12:16 p.m.)


Review request for drill.


Changes
-------

Revise code based review comments.


Repository: drill-git


Description
-------

See https://issues.apache.org/jira/browse/DRILL-2236#.


Diffs (updated)
-----

  
exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/HashJoinPrel.java
 a3c42de 
  
exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/JoinPrel.java
 3541db7 
  
exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/MergeJoinPrel.java
 394a82c 
  
exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/PlannerSettings.java
 faa8546 
  
exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/explain/NumberingRelWriter.java
 6522ad9 
  
exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/visitor/SwapHashJoinVisitor.java
 PRE-CREATION 
  
exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/DefaultSqlHandler.java
 0ac7c97 
  
exec/java-exec/src/main/java/org/apache/drill/exec/server/options/SystemOptionManager.java
 aa0a5ad 

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


Testing
-------

Unit test done. 

Other regression workloads are running. Will update status once complete.


Thanks,

Jinfeng Ni

Reply via email to