Apache9 commented on code in PR #7368:
URL: https://github.com/apache/hbase/pull/7368#discussion_r2413746730
##########
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RegionCoprocessorHost.java:
##########
@@ -1426,6 +1426,39 @@ public void call(RegionObserver observer) throws
IOException {
});
}
+ /**
+ * Supports Coprocessor 'bypass'.
+ * @return true if default behavior should be bypassed, false otherwise
+ * @deprecated Since hbase-2.0.0. No replacement. To be removed in
hbase-3.0.0 and replaced with
+ * something that doesn't expose IntefaceAudience.Private
classes.
+ */
+ @Deprecated
+ public boolean preWALRestore(final RegionInfo info, final WALKey logKey,
final WALEdit logEdit)
+ throws IOException {
+ return execOperation(
+ coprocEnvironments.isEmpty() ? null : new
RegionObserverOperationWithoutResult(true) {
+ @Override
+ public void call(RegionObserver observer) throws IOException {
+ observer.preWALRestore(this, info, logKey, logEdit);
+ }
+ });
+ }
+
+ /**
+ * @deprecated Since hbase-2.0.0. No replacement. To be removed in
hbase-3.0.0 and replaced with
Review Comment:
Ditto.
##########
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RegionCoprocessorHost.java:
##########
@@ -1426,6 +1426,39 @@ public void call(RegionObserver observer) throws
IOException {
});
}
+ /**
+ * Supports Coprocessor 'bypass'.
+ * @return true if default behavior should be bypassed, false otherwise
+ * @deprecated Since hbase-2.0.0. No replacement. To be removed in
hbase-3.0.0 and replaced with
Review Comment:
I think we could remove the deprecation.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
