For most cases, the host and port are needed to declare where the server is. 
However in a special case, we need to not have the port since we are using the 
data to build a service principal name.   

I guess that I can break this into two methods. I originally did this and then 
changed it. I figured someone would see the getAdminServerHost method and not 
notice the getAdminServerHostAndPort method or not know to get the port using 
some getAdminServerPort method.  Also since most of the time the hostname and 
port values were to be used together, I would have that value cached and ready 
to go rather then recalculating the string every time. 


[ Full content available at: https://github.com/apache/ambari/pull/2147 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to