[ https://issues.apache.org/jira/browse/ACCUMULO-1393?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Corey J. Nolet updated ACCUMULO-1393: ------------------------------------- Comment: was deleted (was: Just to be clear, are we referring to the this method in the ServerServices? {code} // not necessary: everything should be advertizing ports in zookeeper int getDefaultPort() { switch (this) { case TSERV_CLIENT: return AccumuloConfiguration.getDefaultConfiguration().getPort(Property.TSERV_CLIENTPORT); case MASTER_CLIENT: return AccumuloConfiguration.getDefaultConfiguration().getPort(Property.MASTER_CLIENTPORT); case GC_CLIENT: return AccumuloConfiguration.getDefaultConfiguration().getPort(Property.GC_PORT); default: throw new IllegalArgumentException(); } } {code} As well as the corresponding code in Property that is setting a default port for the TSERVER_CLIENTPORT and MASTER_CLIENTPORT? ) > Removed code related to default ports in client code > ---------------------------------------------------- > > Key: ACCUMULO-1393 > URL: https://issues.apache.org/jira/browse/ACCUMULO-1393 > Project: Accumulo > Issue Type: Improvement > Reporter: Keith Turner > Assignee: Corey J. Nolet > Fix For: 1.6.0 > > > The Accumulo client code has code for dealing with default master and tserver > ports. This code is old and unused. Its always expected that locations > listed in zookeeper and the metadata table will have a ip+port now. If this > expectation is not met, it should be treated as an error instead of falling > back to a default. Removing this code would clean up the client and logging > produced by the client. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira