You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tinkerpop.apache.org by sp...@apache.org on 2016/12/29 15:27:47 UTC

tinkerpop git commit: TINKERPOP-1130 Added Request/ResponseMessage incompatibilty reasons.

Repository: tinkerpop
Updated Branches:
  refs/heads/TINKERPOP-1130 70c5920ea -> fe9221151


TINKERPOP-1130 Added Request/ResponseMessage incompatibilty reasons.


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

Branch: refs/heads/TINKERPOP-1130
Commit: fe922115167406947f6d4b6ea0f9d0b82eed6c46
Parents: 70c5920
Author: Stephen Mallette <sp...@genoprime.com>
Authored: Thu Dec 29 10:27:01 2016 -0500
Committer: Stephen Mallette <sp...@genoprime.com>
Committed: Thu Dec 29 10:27:01 2016 -0500

----------------------------------------------------------------------
 .../tinkerpop/gremlin/structure/io/Model.java   | 32 +++++++++-----------
 1 file changed, 14 insertions(+), 18 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/fe922115/gremlin-tools/gremlin-io-test/src/main/java/org/apache/tinkerpop/gremlin/structure/io/Model.java
----------------------------------------------------------------------
diff --git a/gremlin-tools/gremlin-io-test/src/main/java/org/apache/tinkerpop/gremlin/structure/io/Model.java b/gremlin-tools/gremlin-io-test/src/main/java/org/apache/tinkerpop/gremlin/structure/io/Model.java
index 3ecaccb..3bf7770 100644
--- a/gremlin-tools/gremlin-io-test/src/main/java/org/apache/tinkerpop/gremlin/structure/io/Model.java
+++ b/gremlin-tools/gremlin-io-test/src/main/java/org/apache/tinkerpop/gremlin/structure/io/Model.java
@@ -332,32 +332,22 @@ public class Model {
         addEntry("Graph Process", obj, title, description);
     }
 
-    private void addGraphProcessEntry(final Object obj, final String title, final String description, final Compatibility... incompatibleWith) {
-        addGraphProcessEntry(obj, title, description, null, incompatibleWith);
-    }
-
     private void addGraphProcessEntry(final Object obj, final String title, final String description, final Map<Compatibility, String> incompatibilityNotes, final Compatibility... incompatibleWith) {
         addEntry("Graph Process", obj, title, description, incompatibilityNotes, incompatibleWith);
     }
 
-    private void addGraphProcessEntry(final Object obj, final String title, final String description, final List<Compatibility> compatibleWith) {
-        addEntry("Graph Process", obj, title, description, compatibleWith);
-    }
-
-    private void addRequestMessageEntry(final Object obj, final String title, final String description) {
-        addEntry("RequestMessage", obj, title, description, GRAPHSON_ONLY);
-    }
-
     private void addRequestMessageEntry(final Object obj, final String title, final String description, final Compatibility... incompatibleWith) {
-        addEntry("RequestMessage", obj, title, description, incompatibleWith);
-    }
-
-    private void addResponseMessageEntry(final Object obj, final String title, final String description) {
-        addEntry("ResponseMessage", obj, title, description, GRAPHSON_ONLY);
+        addEntry("RequestMessage", obj, title, description,
+                createIncompatibilityMap("RequestMessage is not testable prior to Gryo 3.0 as serialization was handled by an intermediate component (MessageSerializer) that doesn't fit the test model.",
+                        GryoCompatibility.V1D0_3_2_3, GryoCompatibility.V1D0_3_2_4, GryoCompatibility.V1D0_3_3_0),
+                incompatibleWith);
     }
 
     private void addResponseMessageEntry(final Object obj, final String title, final String description, final Compatibility... incompatibleWith) {
-        addEntry("ResponseMessage", obj, title, description, incompatibleWith);
+        addEntry("ResponseMessage", obj, title, description,
+                createIncompatibilityMap("ResponseMessage is not testable prior to Gryo 3.0 as serialization was handled by an intermediate component (MessageSerializer) that doesn't fit the test model.",
+                        GryoCompatibility.V1D0_3_2_3, GryoCompatibility.V1D0_3_2_4, GryoCompatibility.V1D0_3_3_0),
+                incompatibleWith);
     }
 
     private void addExtendedEntry(final Object obj, final String title) {
@@ -417,6 +407,12 @@ public class Model {
         entries.get(group).add(new Entry(title, obj, description, compatibleWith, maker, incompatibilityNotes));
     }
 
+    private Map<Compatibility, String> createIncompatibilityMap(final String msg, final Compatibility... incompatibilities) {
+        final Map<Compatibility, String> m = new HashMap<>();
+        Arrays.asList(incompatibilities).forEach(c -> m.put(c, msg));
+        return m;
+    }
+
     public void saveAsCsv(final String file) throws Exception {
         final File f = new File(file);
         f.getParentFile().mkdirs();