kerneltime commented on PR #10485: URL: https://github.com/apache/ozone/pull/10485#issuecomment-4687876136
Done in the latest force-push. Collapsed both `createRaftPeer` overloads into the single form you suggested, and updated `newOMRatisServer` to iterate `peerNodes.values()` directly. Also reworded the Javadoc and the SCM-side comments per Copilot's earlier feedback: the invariant is now described as "pass the configured host string through verbatim; don't pre-resolve into an `InetSocketAddress`" rather than "must be hostname". IP-literal configs are explicitly noted as still honored. -- 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]
