I was trying to use HostPort in some voter / config-tool refac.
Here are some further changes regarding HostPort to move the parse() methods to the HostPort class in order to avoid to deal with the utility class each time we parse a HostPort.
Note: I specifically changes the method names to break compatibility in order to change the client code accordingly, in order to be able to remove the default values set for the ports.
It is up to the caller to know which port fits.
2 flavors of parse methods now:
parse(String, int) => takes a default port in case none is supplied
parse(String) => expect a port to be there (i.e. case for deserialisers for example)
I was trying to use HostPort in some voter / config-tool refac.
Here are some further changes regarding
HostPort
to move the parse() methods to the HostPort class in order to avoid to deal with the utility class each time we parse a HostPort.Note: I specifically changes the method names to break compatibility in order to change the client code accordingly, in order to be able to remove the default values set for the ports.
It is up to the caller to know which port fits.
2 flavors of parse methods now: