F64116045 commented on PR #10928: URL: https://github.com/apache/ozone/pull/10928#issuecomment-5158711570
While reviewing the allocation path, I noticed one related point and would like to check whether this is already tracked somewhere. `recordPendingAllocation(...)` is defined on `PipelineManager` / `PipelineManagerImpl`, but I could not find a caller on the `HDDS-11233` branch. From what I found, container allocation calls `hasEnoughSpace(...)` from `ContainerManagerImpl`, but I do not see a matching `recordPendingAllocation(...)` call after successful allocation. I may be missing some context here. If this is not already covered by another task, would it make sense to track it as a follow-up? I’m happy to help with it as well. -- 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]
