Source:
<code>public static void init(VolumeManager fs, ServerConfigurationFactory serverConfig, String application) throws IOException { final AccumuloConfiguration conf = serverConfig.getConfiguration(); final Instance instance = serverConfig.getInstance(); final String logConfigFile = locateLogConfig(System.getenv("ACCUMULO_CONF_DIR"), application); MonitorLog4jWatcher logConfigWatcher = new MonitorLog4jWatcher(instance.getInstanceID(), logConfigFile); logConfigWatcher.setDelay(5000L); logConfigWatcher.start(); <START> int logPort[] = conf.getPort(Property.MONITOR_LOG4J_PORT); <END> System.setProperty("org.apache.accumulo.core.host.log.port", Integer.toString(logPort[0])); log.info(application + " starting"); log.info("Instance " + serverConfig.getInstance().getInstanceID()); int dataVersion = Accumulo.getAccumuloPersistentVersion(fs); log.info("Data Version " + dataVersion); Accumulo.waitForZookeeperAndHdfs(fs); Version codeVersion = new Version(Constants.VERSION); if (!(canUpgradeFromDataVersion(dataVersion))) { throw new RuntimeException("This version of accumulo (" + codeVersion + ") is not compatible with files stored using data version " + dataVersion); } TreeMap<String,String> sortedProps = new TreeMap<String,String>(); for (Entry<String,String> entry : conf) sortedProps.put(entry.getKey(), entry.getValue()); for (Entry<String,String> entry : sortedProps.entrySet()) { String key = entry.getKey(); log.info(key + " = " + (Property.isSensitive(key) ? "<hidden>" : entry.getValue())); } monitorSwappiness(conf); final String SSL = "SSL"; for (Property sslProtocolProperty : Arrays.asList(Property.RPC_SSL_CLIENT_PROTOCOL, Property.RPC_SSL_ENABLED_PROTOCOLS, Property.MONITOR_SSL_INCLUDE_PROTOCOLS)) { String value = conf.get(sslProtocolProperty); if (value.contains(SSL)) { log.warn("It is recommended that " + sslProtocolProperty + " only allow TLS"); } } }</code><technical_language>put brackets type instead of variable? This is form is valid [discouraged](<LINK_0> This occurs a couple times in PR</technical_language>

Target:
public static void init(VolumeManager fs, ServerConfigurationFactory serverConfig, String application) throws IOException { final AccumuloConfiguration conf = serverConfig.getConfiguration(); final Instance instance = serverConfig.getInstance(); final String logConfigFile = locateLogConfig(System.getenv("ACCUMULO_CONF_DIR"), application); MonitorLog4jWatcher logConfigWatcher = new MonitorLog4jWatcher(instance.getInstanceID(), logConfigFile); logConfigWatcher.setDelay(5000L); logConfigWatcher.start(); int[] logPort = conf.getPort(Property.MONITOR_LOG4J_PORT); System.setProperty("org.apache.accumulo.core.host.log.port", Integer.toString(logPort[0])); log.info(application + " starting"); log.info("Instance " + serverConfig.getInstance().getInstanceID()); int dataVersion = Accumulo.getAccumuloPersistentVersion(fs); log.info("Data Version " + dataVersion); Accumulo.waitForZookeeperAndHdfs(fs); Version codeVersion = new Version(Constants.VERSION); if (!(canUpgradeFromDataVersion(dataVersion))) { throw new RuntimeException("This version of accumulo (" + codeVersion + ") is not compatible with files stored using data version " + dataVersion); } TreeMap<String,String> sortedProps = new TreeMap<String,String>(); for (Entry<String,String> entry : conf) sortedProps.put(entry.getKey(), entry.getValue()); for (Entry<String,String> entry : sortedProps.entrySet()) { String key = entry.getKey(); log.info(key + " = " + (Property.isSensitive(key) ? "<hidden>" : entry.getValue())); } monitorSwappiness(conf); final String SSL = "SSL"; for (Property sslProtocolProperty : Arrays.asList(Property.RPC_SSL_CLIENT_PROTOCOL, Property.RPC_SSL_ENABLED_PROTOCOLS, Property.MONITOR_SSL_INCLUDE_PROTOCOLS)) { String value = conf.get(sslProtocolProperty); if (value.contains(SSL)) { log.warn("It is recommended that " + sslProtocolProperty + " only allow TLS"); } } }