DATAES-316 - Revert: NodeClientFactoryBean doesn't shutdown ES on destroy life-cycle.

Original PR: #335

Author: sothawo

src/main/java/org/springframework/data/elasticsearch/client/NodeClientFactoryBean.java

@@ -152,9 +152,12 @@ public void destroy() throws Exception {
 		try {
 			// NodeClient.close() is a noop, no need to call it here
 			logger.info("Closing elasticSearch node");
-			if (node != null) {
-				node.close();
-			}
+			// closing the Node causes tests to fail in the pipeline build and on some
+			// "mvn test" runs, there seem still to be tests relying on a running instance that
+			// is not properly set up.
+//			if (node != null) {
+//				node.close();
+//			}
 		} catch (final Exception e) {
 			logger.error("Error closing ElasticSearch client: ", e);
 		}