nodece opened a new pull request, #4588: URL: https://github.com/apache/bookkeeper/pull/4588
### Motivation The existing implementation resolves the default hostname for the network interface early, even when it's not required. This can lead to unnecessary DNS resolution or even failures in environments where hostname mappings are missing or incomplete. The goal is to simplify and make host address resolution more robust by: - Using the default IP address as the starting point. - Performing reverse DNS lookup (IP → hostname) only when `useHostNameAsBookieID` is enabled. - Avoiding unnecessary dependency on hostname resolution when IP-based Bookie IDs are sufficient. ### Changes - Replaced `DNS.getDefaultHost(iface)` with `DNS.getDefaultIP(iface)` as the default starting point. - Introduced conditional logic to resolve the hostname from the IP only when `useHostNameAsBookieID` is true. - Simplified code structure and reduced redundant resolution steps. -- 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]
