Github user vrozov commented on a diff in the pull request:
https://github.com/apache/drill/pull/1110#discussion_r167089878
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/base/PhysicalVisitor.java
---
@@ -55,10 +56,10 @@
public RETURN visitExchange(Exchange exchange, EXTRA value) throws EXCEP;
+ public RETURN visitSingleMergeExchange(SingleMergeExchange exchange,
EXTRA value) throws EXCEP;
--- End diff --
The same question as for `PrelVisitor.java`. Is it necessary to have
separate `visitSingleMergeExchange`?
---
