Apache9 commented on a change in pull request #2953:
URL: https://github.com/apache/hbase/pull/2953#discussion_r578194597
##########
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/coprocessor/RegionObserver.java
##########
@@ -968,12 +968,34 @@ default CheckAndMutateResult
postCheckAndMutate(ObserverContext<RegionCoprocesso
* @param c the environment provided by the region server
* @param append Append object
* @return result to return to the client if bypassing default processing
+ * @deprecated since 3.0.0 and will be removed in 4.0.0. Use
+ * {@link #preAppend(ObserverContext, Append, WALEdit, Durability)}
instead.
*/
+ @Deprecated
default Result preAppend(ObserverContext<RegionCoprocessorEnvironment> c,
Append append)
- throws IOException {
+ throws IOException {
return null;
}
+ /**
+ * Called before Append.
+ * <p>
+ * Call CoprocessorEnvironment#bypass to skip default actions.
+ * If 'bypass' is set, we skip out on calling any subsequent chained
coprocessors.
+ * <p>
+ * Note: Do not retain references to any Cells in 'append' beyond the life
of this invocation.
+ * If need a Cell reference for later use, copy the cell and use that.
+ * @param c the environment provided by the region server
+ * @param append Append object
+ * @param edit The WALEdit object that will be written to the wal
+ * @param durability Persistence guarantee for this Put
Review comment:
I suggest we remove it. It will confuse others.
Maybe we should also remove the durability parameter of the other methods if
it is just the same with the ones in the mutation?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
