diff --git a/buildSrc/src/main/resources/checkstyle_suppressions.xml b/buildSrc/src/main/resources/checkstyle_suppressions.xml index f52961553fb0..cb0b87c49c33 100644 --- a/buildSrc/src/main/resources/checkstyle_suppressions.xml +++ b/buildSrc/src/main/resources/checkstyle_suppressions.xml @@ -48,7 +48,6 @@ - diff --git a/server/src/main/java/org/elasticsearch/node/Node.java b/server/src/main/java/org/elasticsearch/node/Node.java index 70ee85fa5088..cb9f211e2c0a 100644 --- a/server/src/main/java/org/elasticsearch/node/Node.java +++ b/server/src/main/java/org/elasticsearch/node/Node.java @@ -335,7 +335,8 @@ protected Node( environment.configFile(), Arrays.toString(environment.dataFiles()), environment.logsFile(), environment.pluginsFile()); } - this.pluginsService = new PluginsService(tmpSettings, environment.configFile(), environment.modulesFile(), environment.pluginsFile(), classpathPlugins); + this.pluginsService = new PluginsService(tmpSettings, environment.configFile(), environment.modulesFile(), + environment.pluginsFile(), classpathPlugins); this.settings = pluginsService.updatedSettings(); localNodeFactory = new LocalNodeFactory(settings, nodeEnvironment.nodeId()); @@ -757,11 +758,9 @@ public Node start() throws NodeValidationException { checkIfClusterNameInDataPaths(clusterService.getClusterName(), environment.dataFiles()); - validateNodeBeforeAcceptingRequests(new BootstrapContext(environment, onDiskMetadata), transportService.boundAddress(), pluginsService - .filterPlugins(Plugin - .class) - .stream() - .flatMap(p -> p.getBootstrapChecks().stream()).collect(Collectors.toList())); + validateNodeBeforeAcceptingRequests(new BootstrapContext(environment, onDiskMetadata), transportService.boundAddress(), + pluginsService.filterPlugins(Plugin.class).stream() + .flatMap(p -> p.getBootstrapChecks().stream()).collect(Collectors.toList())); clusterService.addStateApplier(transportService.getTaskManager()); // start after transport service so the local disco is known @@ -863,8 +862,9 @@ private Node stop() { } // During concurrent close() calls we want to make sure that all of them return after the node has completed it's shutdown cycle. - // If not, the hook that is added in Bootstrap#setup() will be useless: close() might not be executed, in case another (for example api) call - // to close() has already set some lifecycles to stopped. In this case the process will be terminated even if the first call to close() has not finished yet. + // If not, the hook that is added in Bootstrap#setup() will be useless: + // close() might not be executed, in case another (for example api) call to close() has already set some lifecycles to stopped. + // In this case the process will be terminated even if the first call to close() has not finished yet. @Override public synchronized void close() throws IOException { if (lifecycle.started()) {