[
https://issues.apache.org/jira/browse/TINKERPOP-3023?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17974012#comment-17974012
]
ASF GitHub Bot commented on TINKERPOP-3023:
-------------------------------------------
spmallette commented on code in PR #3133:
URL: https://github.com/apache/tinkerpop/pull/3133#discussion_r2145920289
##########
docs/src/recipes/between-vertices.asciidoc:
##########
@@ -22,14 +22,14 @@ traversal on the paths found between them. Consider the
following examples using
[gremlin-groovy,modern]
----
-g.V(1).bothE() <1>
-g.V(1).bothE().where(otherV().hasId(2)) <2>
+g.V(1).bothE() <1>
+g.V(1).bothE().where(otherV().hasId(2)) <2>
v1 = g.V(1).next();[]
v2 = g.V(2).next();[]
-g.V(v1).bothE().where(otherV().is(v2)) <3>
-g.V(v1).outE().where(inV().is(v2)) <4>
-g.V(1).outE().where(inV().has(id, within(2,3))) <5>
-g.V(1).out().where(__.in().hasId(6)) <6>
+g.V(v1.id()).bothE().where(otherV().id().is(v2.id())) <3>
Review Comment:
+1 to not removing the sugar syntax. at some point we'll need to better
acknowledge sugar syntax vs what the grammar supports. it would be nice to
come up with some foundations for that here.
> Expand type syntax in grammar in 3.8
> ------------------------------------
>
> Key: TINKERPOP-3023
> URL: https://issues.apache.org/jira/browse/TINKERPOP-3023
> Project: TinkerPop
> Issue Type: Improvement
> Components: language
> Affects Versions: 3.8.0
> Reporter: Stephen Mallette
> Priority: Major
>
> There are certain types commonly used in Gremlin that should have some native
> support in the grammar:
> * UUID
> ** {{UUID()}} - random
> ** {{UUID('1e077e63-e45a-4f8e-aa00-9b6ffd91f20e')}}
> * Edge
> ** {{Edge(11)}} - different than current syntax for Vertex, adjust Vertex to
> match - drop grammar support for {{{}ReferenceVertex{}}}.
> * Binary (ByteBuffer)
> ** {{Binary( '/9j/4AAQSkZJRgABAQEAAAAAAAD/==')}}
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
