Github user JamesRTaylor commented on a diff in the pull request:
https://github.com/apache/phoenix/pull/239#discussion_r118314856
--- Diff:
phoenix-core/src/main/java/org/apache/phoenix/iterate/RegionContext.java ---
@@ -0,0 +1,15 @@
+package org.apache.phoenix.iterate;
+
+import org.apache.hadoop.hbase.coprocessor.RegionCoprocessorEnvironment;
+
+/**
+ * Maintains coprocessor environment state to
+ * make region observer functionality work for
+ * non-coprocessor environment
+ */
+public interface RegionContext {
+
+ /** @return the Coprocessor environment */
+ RegionCoprocessorEnvironment getEnvironment();
--- End diff --
If this is the only method, we don't really need this interface. Why not
just keep using RegionCoprocessorEnvironment in our APIs and you can just
instantiate you snapshot-based one prior to getting the RegionScanner?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
