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/26 14:18:53 UTC

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

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

Branch: refs/heads/TINKERPOP-1389
Commit: 508e1fd3f9b17f3f57e78c607de20c20a8e0b9c4
Parents: 6a6fa7b
Author: Marko A. Rodriguez <ok...@gmail.com>
Authored: Mon Oct 24 13:31:58 2016 -0600
Committer: Marko A. Rodriguez <ok...@gmail.com>
Committed: Wed Oct 26 08:14:21 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/508e1fd3/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) {