mikepersonick commented on a change in pull request #1551:
URL: https://github.com/apache/tinkerpop/pull/1551#discussion_r787730911



##########
File path: 
gremlin-dotnet/test/Gremlin.Net.IntegrationTest/Gherkin/ScenarioData.cs
##########
@@ -133,6 +137,70 @@ public ScenarioData(IMessageSerializer messageSerializer)
             }
         }
 
+        private static IDictionary<string, VertexProperty> 
GetVertexProperties(GraphTraversalSource g)
+        {
+            try
+            {
+                /*
+                 * This closure will turn a VertexProperty into a triple 
string of the form:
+                 * "vertexName-propKey->propVal"
+                 *
+                 * It will also take care of wrapping propVal in the 
appropriate Numeric format. We must do this in
+                 * case a Vertex has multiple properties with the same key and 
number value but in different numeric
+                 * type spaces (rare, but possible, and presumably something 
we may want to write tests around).
+                 */
+                string groovy = @"
+                    { vp ->
+                          def result = vp.element().value('name') + '-' + 
vp.key() + '->'
+                          def value = vp.value()
+                          def type = ''
+                          switch(value) {

Review comment:
       It was the same in Java. Now some poor soul has to do the same in 
Javascript and Python...




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to