errose28 opened a new pull request #2354: URL: https://github.com/apache/ozone/pull/2354
## What changes were proposed in this pull request? HDDS-5252 added a check that datanodes have enough space for Ratis factor three pipelines. This PR adds this check for Ratis factor one pipelines as well. These factor one pipelines could be auto created based on the configuration, or written to using the replication factor command line flag. Some refactoring of the original approach was done to make this easier to follow IMHO. Previously the existing `sizeRequired` parameter of `PlacementPolicy#chooseDatanodes` was ignored in pipeline policies, and only used by container placement policies. Since we are now placing a size requirement on pipeline creation, it makes sense to use this existing parameter instead of always passing 0 and having the implementation ignore the value while pulling a separate value from the configuration. ## What is the link to the Apache JIRA HDDS-5337 ## How was this patch tested? Unit test added. Existing unit test updated. -- 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]
