szetszwo commented on PR #9553: URL: https://github.com/apache/ozone/pull/9553#issuecomment-3869285845
> Are you saying we have separate implementation and make it configurable? Configurable is a good idea: - People not using snapshot can use RDBBatchOperation. - Even people using snapshot could choose to use RDBBatchOperation (DeleteRange can be implemented iterating Delete). This is very useful in case there is a bug in the new code. > Then it would mean we would have to branch the logic OmResponse where it would have to understand which implementation has been passed No. We could a RDBBatchOperationFactory. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
