Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/1045#discussion_r156181112
--- 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;
--- End diff --
You are a brave man. I elected not to do this renaming because I didn't
think I'd get anyone to review it. Thanks for doing it so I can do the review
:-)
---
