F64116045 commented on PR #10911: URL: https://github.com/apache/ozone/pull/10911#issuecomment-5142321452
Thanks @hevinhsu for adding the context! Yes, I think this could affect custom `DNSToSwitchMapping` implementations that relied on SCM's wrapper for hostname normalization or caching. My understanding is that the proposed behavior is to respect each configured `DNSToSwitchMapping` implementation: if an implementation needs caching, it can either implement caching itself or use `CachedDNSToSwitchMapping` where its normalization behavior is also desired. I agree this compatibility point should be confirmed by the maintainers. For OM, if this behavior change is accepted, I’m happy to help with the OM side as well, either in this PR or as a follow-up. -- 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]
