jojochuang commented on PR #10122: URL: https://github.com/apache/ozone/pull/10122#issuecomment-4997170775
@smengcl Thanks for the July 7 review. Both items are addressed in subsequent commits: 1. **notifyStatusChanged counter reset** — `reconstructionCommandIdToPendingFragmentCount` and `inflightReconstructionCount` are cleared when pending ops are cleared on leader transition (`fd8f55ca459` / `61e4eac3650`). 2. **Processor break halting all under-replication** — enforcement was moved from `UnhealthyReplicationProcessor` to `sendThrottledReconstructionCommand()` so 1-1 replication continues when the reconstruction cap is reached (`61e4eac3650`). Latest push (`aad8da6d69d`) also: - always tracks inflight reconstruction count (even when limit is disabled) for observability and runtime reconfig - uses atomic CAS reservation before send (`c196dd48a0f`) - updates the PR description to match send-path enforcement All review threads are resolved. Could you take another look when you have a chance? -- 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]
