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

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

spmallette commented on pull request #1173: TINKERPOP-2133 TextP support in 
Neo4j
URL: https://github.com/apache/tinkerpop/pull/1173
 
 
   https://issues.apache.org/jira/browse/TINKERPOP-2133
   
   In addition to the primary changes of this issue related to `TextP`, this PR 
also removes the experimental (now deprecated) multi/meta-property support we 
had in `Neo4jGraph`. Removing that stuff and related testing sped up the build 
time for `neo4j-graph` dramatically. 
   
   All tests pass with `docker/build.sh -t -n -i`
   
   VOTE +1
 
----------------------------------------------------------------
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:
[email protected]


> Use neo4j index lookup in Neo4jGraphStep with HasContainers containing TextP 
> predicates
> ---------------------------------------------------------------------------------------
>
>                 Key: TINKERPOP-2133
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-2133
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: neo4j
>    Affects Versions: 3.4.0
>            Reporter: Andrey Skorikov
>            Priority: Trivial
>
> When evaluating a Neo4jGraphStep with HasContainers containing TextP 
> predicates, for example: g.V().has("Label", "name", 
> TextP.containing("substring")), a scan over all vertices with the label is 
> performed.
> Currently, an index lookup is used only when an complete property value is 
> given, that is g.V().has("Label", "name", "exact") - implemented 
> [here|https://github.com/apache/tinkerpop/blob/master/neo4j-gremlin/src/main/java/org/apache/tinkerpop/gremlin/neo4j/structure/trait/NoMultiNoMetaNeo4jTrait.java#L172].
> Allowing to use an index lookup for TextP predicates like containing, 
> startingWith would substantially improve the evaluation performance.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)

Reply via email to