@dalaro provided an update to KryoShimServiceLoader via a gist as the LOG 
message was not so clear.


Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/cafeefce
Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/cafeefce
Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/cafeefce

Branch: refs/heads/master
Commit: cafeefce8a112445cda6930ba3c51eb9a4dc5ef2
Parents: ef18e4e
Author: Marko A. Rodriguez <[email protected]>
Authored: Mon Oct 24 13:31:58 2016 -0600
Committer: Marko A. Rodriguez <[email protected]>
Committed: Tue Nov 29 04:54:21 2016 -0700

----------------------------------------------------------------------
 .../structure/io/gryo/kryoshim/KryoShimServiceLoader.java    | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/cafeefce/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/gryo/kryoshim/KryoShimServiceLoader.java
----------------------------------------------------------------------
diff --git 
a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/gryo/kryoshim/KryoShimServiceLoader.java
 
b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/gryo/kryoshim/KryoShimServiceLoader.java
index 4227edc..c026130 100644
--- 
a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/gryo/kryoshim/KryoShimServiceLoader.java
+++ 
b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/gryo/kryoshim/KryoShimServiceLoader.java
@@ -19,8 +19,6 @@
 package org.apache.tinkerpop.gremlin.structure.io.gryo.kryoshim;
 
 import org.apache.commons.configuration.Configuration;
-import org.apache.tinkerpop.shaded.kryo.io.Input;
-import org.apache.tinkerpop.shaded.kryo.io.Output;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -106,6 +104,9 @@ public class KryoShimServiceLoader {
 
             if (0 != services.size()) {
                 result = services.get(services.size() - 1);
+
+                log.info("Set {} provider to {} ({}) because its priority 
value ({}) is the best available",
+                        KryoShimService.class.getSimpleName(), result, 
result.getClass(), result.getPriority());
             }
         }
 
@@ -114,9 +115,6 @@ public class KryoShimServiceLoader {
             throw new IllegalStateException("Unable to load KryoShimService");
         }
 
-        log.info("Set {} provider to {} ({}) because its priority value ({}) 
is the highest available",
-                KryoShimService.class.getSimpleName(), result, 
result.getClass(), result.getPriority());
-
         final Configuration userConf = conf;
 
         if (null != userConf) {

Reply via email to