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

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

thevalzo commented on issue #1237:
URL: https://github.com/apache/tinkerpop/pull/1237#issuecomment-618411309


   Yes, but you said previously:
   "It made me wonder if we should try to simply optimize match() better so 
that it gives providers a better chance at using an index...in that way we get 
improvements to match() for native usage and to sparql-gremlin and other 
language compilers. As I mentioned previously, we'll create new JIRA issues as 
needed for these improvements going forward."
   
   Since the index optimization of Gremlin's match() improve both native usage 
and sparql-gremlin, I was wondering if something has been done about it.
   


----------------------------------------------------------------
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.

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


> Generate traversals that will better yield index lookups with SPARQL
> --------------------------------------------------------------------
>
>                 Key: TINKERPOP-2325
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-2325
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: sparql
>    Affects Versions: 3.4.4
>            Reporter: Stephen Mallette
>            Assignee: Stephen Mallette
>            Priority: Major
>             Fix For: 3.5.0, 3.4.5
>
>
> The following simple bit of SPARQL generates a {{match()}} that wont easily 
> be converted to further utilize indices. 
> {code}
> gremlin> s.sparql("SELECT ?x WHERE { ?x v:name 'marko' }").explain()
> ==>Traversal Explanation
> ===========================================================================================================================================================================================
> Original Traversal                 [InjectStep([SELECT ?x WHERE { ?x v:name 
> 'marko' }])]
> ConnectiveStrategy           [D]   [InjectStep([SELECT ?x WHERE { ?x v:name 
> 'marko' }])]
> SparqlStrategy               [D]   [GraphStep(vertex,[]), 
> MatchStep(AND,[[MatchStartStep(x), PropertiesStep([name],value), 
> IsStep(eq(marko)), MatchEndStep]]), SelectOneStep(last,x)]
> MatchPredicateStrategy       [O]   [GraphStep(vertex,[]), 
> MatchStep(AND,[[MatchStartStep(x), PropertiesStep([name],value), 
> IsStep(eq(marko)), MatchEndStep]]), SelectOneStep(last,x)]
> FilterRankingStrategy        [O]   [GraphStep(vertex,[]), 
> MatchStep(AND,[[MatchStartStep(x), PropertiesStep([name],value), 
> IsStep(eq(marko)), MatchEndStep]]), SelectOneStep(last,x)]
> EarlyLimitStrategy           [O]   [GraphStep(vertex,[]), 
> MatchStep(AND,[[MatchStartStep(x), PropertiesStep([name],value), 
> IsStep(eq(marko)), MatchEndStep]]), SelectOneStep(last,x)]
> InlineFilterStrategy         [O]   [GraphStep(vertex,[]), 
> MatchStep(AND,[[MatchStartStep(x), PropertiesStep([name],value), 
> IsStep(eq(marko)), MatchEndStep]]), SelectOneStep(last,x)]
> IncidentToAdjacentStrategy   [O]   [GraphStep(vertex,[]), 
> MatchStep(AND,[[MatchStartStep(x), PropertiesStep([name],value), 
> IsStep(eq(marko)), MatchEndStep]]), SelectOneStep(last,x)]
> AdjacentToIncidentStrategy   [O]   [GraphStep(vertex,[]), 
> MatchStep(AND,[[MatchStartStep(x), PropertiesStep([name],value), 
> IsStep(eq(marko)), MatchEndStep]]), SelectOneStep(last,x)]
> RepeatUnrollStrategy         [O]   [GraphStep(vertex,[]), 
> MatchStep(AND,[[MatchStartStep(x), PropertiesStep([name],value), 
> IsStep(eq(marko)), MatchEndStep]]), SelectOneStep(last,x)]
> CountStrategy                [O]   [GraphStep(vertex,[]), 
> MatchStep(AND,[[MatchStartStep(x), PropertiesStep([name],value), 
> IsStep(eq(marko)), MatchEndStep]]), SelectOneStep(last,x)]
> PathRetractionStrategy       [O]   [GraphStep(vertex,[]), 
> MatchStep(AND,[[MatchStartStep(x), PropertiesStep([name],value), 
> IsStep(eq(marko)), MatchEndStep]]), SelectOneStep(last,x)]
> LazyBarrierStrategy          [O]   [GraphStep(vertex,[]), 
> MatchStep(AND,[[MatchStartStep(x), PropertiesStep([name],value), 
> IsStep(eq(marko)), MatchEndStep]]), SelectOneStep(last,x)]
> TinkerGraphCountStrategy     [P]   [GraphStep(vertex,[]), 
> MatchStep(AND,[[MatchStartStep(x), PropertiesStep([name],value), 
> IsStep(eq(marko)), MatchEndStep]]), SelectOneStep(last,x)]
> TinkerGraphStepStrategy      [P]   [TinkerGraphStep(vertex,[]), 
> MatchStep(AND,[[MatchStartStep(x), PropertiesStep([name],value), 
> IsStep(eq(marko)), MatchEndStep]]), SelectOneStep(last,x)]
> ProfileStrategy              [F]   [TinkerGraphStep(vertex,[]), 
> MatchStep(AND,[[MatchStartStep(x), PropertiesStep([name],value), 
> IsStep(eq(marko)), MatchEndStep]]), SelectOneStep(last,x)]
> StandardVerificationStrategy [V]   [TinkerGraphStep(vertex,[]), 
> MatchStep(AND,[[MatchStartStep(x), PropertiesStep([name],value), 
> IsStep(eq(marko)), MatchEndStep]]), SelectOneStep(last,x)]
> Final Traversal                    [TinkerGraphStep(vertex,[]), 
> MatchStep(AND,[[MatchStartStep(x), PropertiesStep([name],value), 
> IsStep(eq(marko)), MatchEndStep]]), SelectOneStep(last,x)]
> {code}
> {{sparql-gremlin}} needs some improvements in this area.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to