ChenSammi commented on PR #8127: URL: https://github.com/apache/ozone/pull/8127#issuecomment-2871367600
> For me, this change is premature optimisation. This is not a problem we have seen and as I said earlier, this change only solves a very small percentage of the potential cases where a container will be deleted right after being moved by the disk balancer. > > I am still against this change being merged, unless we can see evidence of such a problem occurring frequently in practice. Fine, I respect one's right to say no though I don't agree. Didn't expect there will so strong disagreement on this optimization. -- 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: issues-unsubscr...@ozone.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@ozone.apache.org For additional commands, e-mail: issues-h...@ozone.apache.org
