swamirishi commented on PR #8243: URL: https://github.com/apache/ozone/pull/8243#issuecomment-2787772205
> > @swamirishi , thanks for working on this! > > This is a very big change. Could you describe what you are trying to do in some details? What is the algorithm in your mind? The two-line description is way too simple to cover such a large code change. > > @szetszwo Thank you for starting to review the PR. I have updated the PR description with the details about the patch. In the meanwhile I am also working on @errose28's comment on having a single table iterator and just offloading the proto deserialization to the threadpool. While this approach has no issue with consistency but I see some performance implications with this proposal and thus we would need to benchmark both the implementations. Once I have some figures comparing both the implementations I would publish it here. -- 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]
