Repository: tinkerpop Updated Branches: refs/heads/master 8c079598e -> 5529e18cb
Fix error messaging for der/ser request/response messages CTR Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/5529e18c Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/5529e18c Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/5529e18c Branch: refs/heads/master Commit: 5529e18cbeb31d51831063591c5fe47d44398b13 Parents: 8c07959 Author: Stephen Mallette <sp...@genoprime.com> Authored: Fri Feb 9 14:30:03 2018 -0500 Committer: Stephen Mallette <sp...@genoprime.com> Committed: Fri Feb 9 14:30:03 2018 -0500 ---------------------------------------------------------------------- .../driver/ser/AbstractGryoMessageSerializerV3d0.java | 8 ++++---- .../gremlin/driver/ser/GraphSONMessageSerializerV3d0.java | 8 ++++---- 2 files changed, 8 insertions(+), 8 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/5529e18c/gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/ser/AbstractGryoMessageSerializerV3d0.java ---------------------------------------------------------------------- diff --git a/gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/ser/AbstractGryoMessageSerializerV3d0.java b/gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/ser/AbstractGryoMessageSerializerV3d0.java index d74a106..9c690c9 100644 --- a/gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/ser/AbstractGryoMessageSerializerV3d0.java +++ b/gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/ser/AbstractGryoMessageSerializerV3d0.java @@ -155,7 +155,7 @@ public abstract class AbstractGryoMessageSerializerV3d0 extends AbstractMessageS return kryo.readObject(input, ResponseMessage.class); } } catch (Exception ex) { - logger.warn("Response [{}] could not be deserialized by {}.", msg, GryoMessageSerializerV3d0.class.getName()); + logger.warn(String.format("Response [%s] could not be deserialized by %s.", msg, AbstractGryoMessageSerializerV3d0.class.getName()), ex); throw new SerializationException(ex); } } @@ -189,7 +189,7 @@ public abstract class AbstractGryoMessageSerializerV3d0 extends AbstractMessageS } catch (Exception ex) { if (encodedMessage != null) ReferenceCountUtil.release(encodedMessage); - logger.warn("Response [{}] could not be serialized by {}.", responseMessage.toString(), GryoMessageSerializerV3d0.class.getName()); + logger.warn(String.format("Response [%s] could not be serialized by %s.", responseMessage, AbstractGryoMessageSerializerV3d0.class.getName()), ex); throw new SerializationException(ex); } } @@ -206,7 +206,7 @@ public abstract class AbstractGryoMessageSerializerV3d0 extends AbstractMessageS return kryo.readObject(input, RequestMessage.class); } } catch (Exception ex) { - logger.warn("Request [{}] could not be deserialized by {}.", msg, GryoMessageSerializerV3d0.class.getName()); + logger.warn(String.format("Request [%s] could not be deserialized by %s.", msg, AbstractGryoMessageSerializerV3d0.class.getName()), ex); throw new SerializationException(ex); } } @@ -237,7 +237,7 @@ public abstract class AbstractGryoMessageSerializerV3d0 extends AbstractMessageS } catch (Exception ex) { if (encodedMessage != null) ReferenceCountUtil.release(encodedMessage); - logger.warn("Request [{}] could not be serialized by {}.", requestMessage.toString(), GryoMessageSerializerV3d0.class.getName()); + logger.warn(String.format("Request [%s] could not be serialized by %s.", requestMessage, AbstractGryoMessageSerializerV3d0.class.getName()), ex); throw new SerializationException(ex); } } http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/5529e18c/gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/ser/GraphSONMessageSerializerV3d0.java ---------------------------------------------------------------------- diff --git a/gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/ser/GraphSONMessageSerializerV3d0.java b/gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/ser/GraphSONMessageSerializerV3d0.java index 46e5c3a..502a2ff 100644 --- a/gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/ser/GraphSONMessageSerializerV3d0.java +++ b/gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/ser/GraphSONMessageSerializerV3d0.java @@ -78,7 +78,7 @@ public final class GraphSONMessageSerializerV3d0 extends AbstractGraphSONMessage try { return mapper.readValue(msg, ResponseMessage.class); } catch (Exception ex) { - logger.warn("Response [{}] could not be deserialized by {}.", msg, AbstractGraphSONMessageSerializerV2d0.class.getName()); + logger.warn(String.format("Response [%s] could not be deserialized by %s.", msg, GraphSONMessageSerializerV3d0.class.getName()), ex); throw new SerializationException(ex); } } @@ -88,7 +88,7 @@ public final class GraphSONMessageSerializerV3d0 extends AbstractGraphSONMessage try { return mapper.writeValueAsString(responseMessage); } catch (Exception ex) { - logger.warn("Response [{}] could not be serialized by {}.", responseMessage.toString(), AbstractGraphSONMessageSerializerV2d0.class.getName()); + logger.warn(String.format("Response [%s] could not be serialized by %s.", responseMessage.toString(), GraphSONMessageSerializerV3d0.class.getName()), ex); throw new SerializationException(ex); } } @@ -98,7 +98,7 @@ public final class GraphSONMessageSerializerV3d0 extends AbstractGraphSONMessage try { return mapper.readValue(msg, RequestMessage.class); } catch (Exception ex) { - logger.warn("Request [{}] could not be deserialized by {}.", msg, AbstractGraphSONMessageSerializerV2d0.class.getName()); + logger.warn(String.format("Request [%s] could not be deserialized by %s.", msg, GraphSONMessageSerializerV3d0.class.getName()), ex); throw new SerializationException(ex); } } @@ -108,7 +108,7 @@ public final class GraphSONMessageSerializerV3d0 extends AbstractGraphSONMessage try { return mapper.writeValueAsString(requestMessage); } catch (Exception ex) { - logger.warn("Request [{}] could not be serialized by {}.", requestMessage.toString(), AbstractGraphSONMessageSerializerV2d0.class.getName()); + logger.warn(String.format("Request [%s] could not be serialized by %s.", requestMessage.toString(), GraphSONMessageSerializerV3d0.class.getName()), ex); throw new SerializationException(ex); } }