kerneltime commented on PR #10488: URL: https://github.com/apache/ozone/pull/10488#issuecomment-4806933191
Opened a daisy-chained PR for this feature on top of @szetszwo's HDDS-15682 (#10612): https://github.com/szetszwo/ozone/pull/11 On #10612's stable host:port identity, the feature collapses a lot — the new PR drops the `SCMConnectionManager` re-key dance + collision check and the `StateContext` queue migration (all only needed because the key was the resolved IP). Net ~290 lines vs ~1300 here. It also addresses the review nits on #10612 (commit 1) and adds the slimmed refresh trigger + `HostAndPort.refresh()` (commit 2). Leaving this PR (#10488) as-is for now. Next: once #10612 merges to master, retarget the stacked PR's base to master and close this one. (Confirmed #10612 alone does not fix the stale-IP bug — `HostAndPort` resolves once at construction and the RPC proxy freezes it, so the failure-triggered refresh is still required.) Generated-by: Claude Code (AI-assisted) -- 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]
