[jira] [Closed] (TINKERPOP-2714) Ensure that Gremlin translation matches its original form

["Stephen Mallette (Jira)" <ji...@apache.org>]

     [ 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)