swamirishi commented on PR #9553: URL: https://github.com/apache/ozone/pull/9553#issuecomment-3865575156
> @swamirishi , sorry that this does not look like safe change since it dramatically changes RDBBatchOperation. In particular, this PR adds the new `Map<Integer, Op> ops` to support a unneeded case mixing DeleteRange and SingleKeyOp in a batch. > > * I propose support pure batch DeleteRange (but not mixing other SingleKeyOp). The code would be much simpler. That is not even possible. It means double buffer would have to understand how the DBResponse interface is going to flush the transaction on a db. -- 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]
