rich7420 commented on PR #9362: URL: https://github.com/apache/ozone/pull/9362#issuecomment-3581618456
> I'd rather suggest graceful handling of Out of space event from DN during container creation and if it fails to use one of already opened containers. > > The other approach would be serializing container creation if DN has less than 5 -10% of free space. You're right that 2x only provides a buffer for limited concurrency. However, implementing the more robust solutions you suggested (graceful out-of-space handling, serialized allocation) would require significant architectural changes. I think the current approach using the existing `requiredReplicationSpace()` method (2x) provides immediate improvement for common cases while keeping the code simple. So maybe we can consider more comprehensive solutions in future work if this proves insufficient. Thanks the suggestion @yandrey321 ! -- 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]
