shustsud opened a new pull request, #4670:
URL: https://github.com/apache/bookkeeper/pull/4670
### Motivation
Currently, when using `ScriptBasedMapping` for network location resolution
via the topology script, only IP addresses are passed to the script.
In some situations it is preferable to pass hostnames instead of IPs.
This PR introduces a configuration option to enable that behavior.
### Changes
- Added a new configuration key:
`networkTopologyScriptUseHostNameArgs`
- default: `false`
- when set to `true`, `ScriptBasedMapping` passes hostnames instead of IP
addresses to the topology script.
- Updated `CachedDNSToSwitchMapping` and
`ScriptBasedMapping.RawScriptBasedMapping` to respect this setting.
- Added corresponding unit test:
`TestRackawareEnsemblePlacementPolicyUsingScript.testUseHostnameArgsOption`
--
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]
