Merge pull request #4072 from vespa-engine/bjorncs/jdisc-http-service-cleanup

Bjorncs/jdisc http service cleanup

1 files changed, 6 insertions, 3 deletions

diff --git a/jdisc_core/src/main/java/com/yahoo/jdisc/test/ServerProviderConformanceTest.java b/jdisc_core/src/main/java/com/yahoo/jdisc/test/ServerProviderConformanceTest.java
index d2b58090665..e7039e85e5e 100644
--- a/jdisc_core/src/main/java/com/yahoo/jdisc/test/ServerProviderConformanceTest.java
+++ b/jdisc_core/src/main/java/com/yahoo/jdisc/test/ServerProviderConformanceTest.java
@@ -24,7 +24,6 @@ import java.io.StringWriter;
 import java.lang.reflect.InvocationTargetException;
 import java.lang.reflect.Method;
 import java.lang.reflect.Modifier;
-import java.net.URI;
 import java.nio.ByteBuffer;
 import java.util.HashSet;
 import java.util.Set;
@@ -35,14 +34,18 @@ import java.util.concurrent.Executors;
 import java.util.concurrent.RejectedExecutionException;
 import java.util.concurrent.TimeUnit;
 import java.util.concurrent.TimeoutException;
+import java.util.logging.Logger;
 import java.util.stream.Stream;
 
 /**
- * @author <a href="mailto:simon@yahoo-inc.com">Simon Thoresen Hult</a>
+ * @author Simon Thoresen Hult
  */
 @SuppressWarnings("UnusedDeclaration")
 @Beta
 public abstract class ServerProviderConformanceTest {
+
+    private static final Logger log = Logger.getLogger(ServerProviderConformanceTest.class.getName());
+
     private static final int NUM_RUNS_EACH_TEST = 10;
 
     /**
@@ -2790,7 +2793,7 @@ public abstract class ServerProviderConformanceTest {
         serverProvider.release();
 
         for (int i = 0; i < NUM_RUNS_EACH_TEST; ++i) {
-            System.out.println("Test run #" + i);
+            log.fine("Test run #" + i);
             requestHandler.reset(adapter.newResponseContent());
             final U client = adapter.newClient(serverProvider);
             final boolean withRequestContent = requestType == RequestType.WITH_CONTENT;