[ 
https://issues.apache.org/jira/browse/TINKERPOP-2681?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17485860#comment-17485860
 ] 

ASF GitHub Bot commented on TINKERPOP-2681:
-------------------------------------------

spmallette commented on a change in pull request #1555:
URL: https://github.com/apache/tinkerpop/pull/1555#discussion_r797690433



##########
File path: gremlin-dotnet/build/generate.groovy
##########
@@ -31,6 +31,16 @@ import java.nio.file.Paths
 
 import static 
org.apache.tinkerpop.gremlin.process.traversal.AnonymousTraversalSource.traversal
 
+// getting an exception like:
+// > InvocationTargetException: javax.script.ScriptException: 
groovy.lang.MissingMethodException: No signature of
+// > method: 
org.apache.tinkerpop.gremlin.process.traversal.dsl.graph.DefaultGraphTraversal.mergeE()
 is applicable for
+// > argument types: (String) values: [4ffdea36-4a0e-4681-acba-e76875d1b25b]
+// usually means bindings are not being extracted properly by 
VarAsBindingASTTransformation which typically happens
+// when a step is taking an argument that cannot properly resolve to the type 
required by the step itself. there are
+// special cases in that VarAsBindingASTTransformation class which might need 
to be adjusted. Editing the
+// GremlinGroovyScriptEngineTest#shouldProduceBindingsForVars() with the 
failing step and argument can typically make
+// this issue relatively easy to debug and enforce.

Review comment:
       yeah there is a test for it there...i just wanted a comment/reminder 
here in these scripts because i sometimes run afoul of this problem and then 
forget the reason. others would probably be completely lost without this 
pointer.




-- 
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: commits-unsubscr...@tinkerpop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Create merge() step to codify best practice for upsert pattern
> --------------------------------------------------------------
>
>                 Key: TINKERPOP-2681
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-2681
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: language, process
>    Affects Versions: 3.5.1
>            Reporter: Dave Bechberger
>            Assignee: Stephen Mallette
>            Priority: Major
>
> Create a step that codifies the best practice for the upsert functionality 
> into a single step to make it easier to use, more discoverable, and easier 
> for implementers to optimize.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

Reply via email to