adoroszlai commented on pull request #1610: URL: https://github.com/apache/ozone/pull/1610#issuecomment-733127142
> Am I correct in thinking this is mostly a problem with large keys which have many blocks? I think so, and also for `listStatus` with many entries in the result. > Do you think it would make sense to add the sort flag to the `getContainerWithPipeline` and `getContainerWithPipelineBatch` so we can have the pipeline returned sorted with a single RPC call, rather than having to make at least two calls? Actually that's a great idea, better than the third (and not yet implemented) improvement I proposed, which would still require 2 batch requests. I think we have to pay attention to compatibility, though. Client would need to check if SCM supports this new flag, so that it can get sorted pipeline from both new and old SCM. ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
