Apache9 commented on PR #7617: URL: https://github.com/apache/hbase/pull/7617#issuecomment-3777600615
> Why are we targeting this to master? Shouldn't it be on the feature branch? > > Also, as already mentioned by @anmolnar and @taklwu , we should refrain from adding logic that is specific to the continuous backup replication in the generic replication interfaces/classes. This is my suggestion that we should target to master branch to add this feature first, and then reimplement the continuous backup feature on top of this change. Of course, we do not need to merge this to master first, this is just for better reviewing, we can land this to a feature branch and then rebase the continus backup branch. Thanks. -- 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]
