apurtell commented on code in PR #4924:
URL: https://github.com/apache/hbase/pull/4924#discussion_r1061745477


##########
hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSink.java:
##########
@@ -107,6 +107,14 @@ public class ReplicationSink {
   private SourceFSConfigurationProvider provider;
   private WALEntrySinkFilter walEntrySinkFilter;
 
+  /**
+   * If enabled at sink cluster site config, extended WAL attributes would be 
attached as Mutation
+   * attributes. This is useful for source cluster coproc to provide coproc 
specific metadata as WAL
+   * annotations and have them attached back to Mutations generated from WAL 
entries at sink side.
+   */
+  public static final String 
HBASE_REPLICATION_SINK_ATTRIBUTES_WAL_TO_MUTATIONS =

Review Comment:
   This doesn't need to be configurable. Mutation attributes will be ignored if 
they are not expected so this is harmless to clients. And IMHO the right thing 
to be doing. I agree with your premise.
   
   Normally WAL entries will not have attributes, so this is doubly harmless 
for standard use cases. 



-- 
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]

Reply via email to