[ 
https://issues.apache.org/jira/browse/TINKERPOP-2714?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Stephen Mallette closed TINKERPOP-2714.
---------------------------------------
    Fix Version/s: 4.0.0
         Assignee: Stephen Mallette
       Resolution: Fixed

Fixed with new {{Translator}} infrastructure - tested here: 
https://github.com/apache/tinkerpop/blob/38fabbb6b5214a6d6c5b1a0b33bee7a6fee97927/gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/language/translator/GremlinTranslatorTest.java#L922-L930

> Ensure that Gremlin translation matches its original form
> ---------------------------------------------------------
>
>                 Key: TINKERPOP-2714
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-2714
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: process, translator
>    Affects Versions: 3.5.2
>            Reporter: Stephen Mallette
>            Assignee: Stephen Mallette
>            Priority: Major
>             Fix For: 4.0.0
>
>
> It's not clear if there are other places where this is an issue but with the 
> introduction of {{property(Map}}) we've made it so that this call unwinds to 
> {{property(k1,v1).property(k2,v2)...}} which isn't exactly what the user 
> typed. So when you hit translation, it might be confusing to see the latter 
> representation. Not sure if this is a major problem at this time and it seems 
> the value of the new overload from {{property()}} outweighs this concern, but 
> in the longer run we should probably look to ensure the two things remain 
> equivalent.



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

Reply via email to