You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tinkerpop.apache.org by ok...@apache.org on 2016/10/24 19:32:04 UTC

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

Repository: tinkerpop
Updated Branches:
  refs/heads/TINKERPOP-1389 3ebbffaff -> 73dac20a4


@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/73dac20a
Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/73dac20a
Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/73dac20a

Branch: refs/heads/TINKERPOP-1389
Commit: 73dac20a425f843be868d2756b4e894856768f1f
Parents: 3ebbffa
Author: Marko A. Rodriguez <ok...@gmail.com>
Authored: Mon Oct 24 13:31:58 2016 -0600
Committer: Marko A. Rodriguez <ok...@gmail.com>
Committed: Mon Oct 24 13:31:58 2016 -0600

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


http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/73dac20a/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) {