[grid] Removing unused imports, formatting changes and removing CLI default

Author: diemol

java/server/src/org/openqa/selenium/grid/docker/DockerAssetsPath.java

@@ -20,18 +20,9 @@
 import org.openqa.selenium.remote.SessionId;
 
 import java.io.File;
-import java.io.IOException;
-import java.nio.file.Files;
-import java.nio.file.Path;
-import java.nio.file.Paths;
-import java.util.Optional;
-import java.util.logging.Level;
-import java.util.logging.Logger;
 
 public class DockerAssetsPath {
 
-  private static final Logger LOG = Logger.getLogger(DockerAssetsPath.class.getName());
-
   private final String hostPath;
   private final String containerPath;
 

java/server/src/org/openqa/selenium/grid/docker/DockerSessionFactory.java

@@ -114,7 +114,9 @@ public DockerSessionFactory(
   @Override
   public boolean test(Capabilities capabilities) {
     return stereotype.getCapabilityNames().stream()
-        .map(name -> Objects.equals(stereotype.getCapability(name), capabilities.getCapability(name)))
+        .map(
+          name ->
+            Objects.equals(stereotype.getCapability(name), capabilities.getCapability(name)))
         .reduce(Boolean::logicalAnd)
         .orElse(false);
   }
@@ -170,7 +172,9 @@ public Optional<ActiveSession> apply(CreateSessionRequest sessionRequest) {
       try {
         result = new ProtocolHandshake().createSession(client, command);
         response = result.createResponse();
-        attributeMap.put(AttributeKey.DRIVER_RESPONSE.getKey(), EventAttribute.setValue(response.toString()));
+        attributeMap.put(
+          AttributeKey.DRIVER_RESPONSE.getKey(),
+          EventAttribute.setValue(response.toString()));
       } catch (IOException | RuntimeException e) {
         span.setAttribute("error", true);
         span.setStatus(Status.CANCELLED);
@@ -204,8 +208,12 @@ public Optional<ActiveSession> apply(CreateSessionRequest sessionRequest) {
       Dialect downstream = sessionRequest.getDownstreamDialects().contains(result.getDialect()) ?
                            result.getDialect() :
                            W3C;
-      attributeMap.put(AttributeKey.DOWNSTREAM_DIALECT.getKey(), EventAttribute.setValue(downstream.toString()));
-      attributeMap.put(AttributeKey.DRIVER_RESPONSE.getKey(), EventAttribute.setValue(response.toString()));
+      attributeMap.put(
+        AttributeKey.DOWNSTREAM_DIALECT.getKey(),
+        EventAttribute.setValue(downstream.toString()));
+      attributeMap.put(
+        AttributeKey.DRIVER_RESPONSE.getKey(),
+        EventAttribute.setValue(response.toString()));
 
       span.addEvent("Docker driver service created session", attributeMap);
       LOG.fine(String.format(

java/server/src/org/openqa/selenium/grid/log/LoggingFlags.java

@@ -59,7 +59,7 @@ public class LoggingFlags implements HasRoles {
     "Log levels are described here https://docs.oracle.com/javase/7/docs/api/java/util/logging/Level.html ",
     names = "--log-level", arity = 1)
   @ConfigValue(section = "logging", name = "log-level", example = "INFO")
-  private String logLevel = Level.INFO.getName();
+  private String logLevel;
 
   @Override
   public Set<Role> getRoles() {