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

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

Fs02 commented on PR #2127:
URL: https://github.com/apache/tinkerpop/pull/2127#issuecomment-3302496140

   thanks, the type is a bit broken, but I can use that
   ```
   <html>TS7016: Could not find a declaration file for module 
'gremlin/lib/process/graph-traversal'. 
'app/node_modules/gremlin/lib/process/graph-traversal.js' implicitly has an 
'any' type.<br/>If the 'gremlin' package actually exposes this module, consider 
sending a pull request to amend 
'https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/gremlin'
   ```
   
   another question is, is it possible to specify multiple value using 
CardinalityValue.list and use it as `mergeE(...).option(merge.onCreate, 
props).option(merge.onMatch, props)`?
   




> mergeV with sideEffect not correctly updating properties
> --------------------------------------------------------
>
>                 Key: TINKERPOP-2957
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-2957
>             Project: TinkerPop
>          Issue Type: Bug
>          Components: process
>    Affects Versions: 3.6.4
>            Reporter: Kelvin Lawrence
>            Assignee: Stephen Mallette
>            Priority: Blocker
>             Fix For: 3.7.0
>
>
> A Gremlin user let me know about the following issue. If the query below is 
> run twice, the name parameter on the 4567 vertex does not get updated. The 
> name on the 1234 vertex does get updated. The query is written this way as 
> not all databases support single cardinality as the default. The issue has 
> been reproduced using both TinkerGraph and Amazon Neptune.
> {code:java}
> g.mergeV([(T.id): 'test-test-1234']).
>   option(onCreate, [(T.label): 'someLabel', 'name': 'name1']).
>   option(onMatch, sideEffect(property(single,"name","name2")).constant([:])).
>  mergeV([(T.id): 'test-test-4567']).
>   option(onCreate, [(T.label): 'someLabel', 'name': 'name1']).
>   option(onMatch, 
> sideEffect(property(single,"name","name2")).constant([:])){code}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to