Github user vrozov commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1110#discussion_r167088606
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/visitor/PrelVisitor.java
 ---
    @@ -35,7 +38,9 @@
       public RETURN visitScan(ScanPrel prel, EXTRA value) throws EXCEP;
       public RETURN visitJoin(JoinPrel prel, EXTRA value) throws EXCEP;
       public RETURN visitProject(ProjectPrel prel, EXTRA value) throws EXCEP;
    -
    +  public RETURN visitHashToRandomExchange(HashToRandomExchangePrel prel, 
EXTRA value) throws EXCEP;
    --- End diff --
    
    Are 3 new methods necessary? Can `visitExchange` delegate to `prel` or use 
instance of? 


---

Reply via email to