Github user ilooner commented on a diff in the pull request:
https://github.com/apache/drill/pull/1045#discussion_r162521845
--- Diff:
exec/java-exec/src/test/java/org/apache/drill/test/OperatorFixture.java ---
@@ -86,37 +117,35 @@
* <li>Multiple threads of execution.</li>
* </ul>
*/
-
public class OperatorFixture extends BaseFixture implements AutoCloseable {
+ public OperatorContext operatorContext(PhysicalOperator config) {
+ return new MockOperatorContext(context, allocator(), config);
+ }
+
/**
* Builds an operator fixture based on a set of config options and
system/session
* options.
*/
-
- public static class OperatorFixtureBuilder
+ public static class Builder
--- End diff --
It's a nested class in the master branch currently
https://github.com/apache/drill/blob/master/exec/java-exec/src/test/java/org/apache/drill/test/OperatorFixture.java
. So I didn't change that in this PR. Did you want me to pull it out into a
top level class as part of this PR?
---
