rakeshadr commented on PR #10440:
URL: https://github.com/apache/ozone/pull/10440#issuecomment-4647912380
@ashishkumar50 Thanks for the contribution. PR is quite big, could you
please split the PR into logical units.
One idea is to split into two logical units:
```
PR#1: Replication records slots into PendingContainerTracker
- ReplicationManagerUtil.getTargetDatanodes calls
checkSpaceAndRecordAllocation
per candidate before dispatching replication commands
- Rollback slots on CommandTargetOverloadedException /
NotLeaderException
- All four under-replication handler updates
PR#2: PlacementPolicy reads slots from PendingContainerTracker
- hasAvailableSpace API in NodeManager / PendingContainerTracker
- SCMCommonPlacementPolicy.hasEnoughSpace delegates to tracker
- PipelineProvider and NonWritableNodeFilter updates
```
--
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]
