apurtell commented on issue #390: HBASE-22623 - Add RegionObserver coprocessor hook for preWALAppend URL: https://github.com/apache/hbase/pull/390#issuecomment-517818517 > I have not yet incorporated @apurtell 's suggestion to add WALEdit to the coprocessor hook, because of the InterfaceAudience.Private notations on WALEdit.add Change the annotation. Raise it here and on the JIRA for feedback. > I personally don't see an issue with adding WALEdit to the coproc (if it's OK to write a coproc that adds a column to every Put, why not a coproc that adds a Cell to every WALEdit?), but it's not necessary for my purposes, and I don't want to add it in just to find out that I need to take it out after all. I do not want to be in the position in the future, now that we are going to the trouble of adding a hook, to find out that it is not suitable for someone else only because we didn't have the foresight to add WALedit to the method signature. It is impossible to know of course all possible potential future use cases but this one seems somewhat obvious by omission.
---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services
