Github user vrozov commented on a diff in the pull request: https://github.com/apache/drill/pull/1110#discussion_r167586208 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/base/AbstractExchange.java --- @@ -119,7 +119,7 @@ public final void setupReceivers(int majorFragmentId, List<DrillbitEndpoint> rec } @Override - public final <T, X, E extends Throwable> T accept(PhysicalVisitor<T, X, E> physicalVisitor, X value) throws E { + public <T, X, E extends Throwable> T accept(PhysicalVisitor<T, X, E> physicalVisitor, X value) throws E { --- End diff -- Is this change still required?
---