Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/1045#discussion_r156181381
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/ops/BaseOperatorContext.java
---
@@ -39,27 +39,29 @@
* version of the operator context and the full production-time context
* that includes network services.
*/
-
public abstract class BaseOperatorContext implements OperatorContext {
static final org.slf4j.Logger logger =
org.slf4j.LoggerFactory.getLogger(BaseOperatorContext.class);
- protected final FragmentContextInterface context;
+ protected final FragmentContext context;
protected final BufferAllocator allocator;
protected final PhysicalOperator popConfig;
protected final BufferManager manager;
+ protected OperatorStatReceiver statsWriter;
--- End diff --
Didn't I deprecate this in a recent commit? Turned out to be more trouble
than it was worth. The original class could be made to work in a test
environment.
---