[
https://issues.apache.org/jira/browse/TINKERPOP-2133?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16877872#comment-16877872
]
stephen mallette commented on TINKERPOP-2133:
---------------------------------------------
We now have this branch with the work partially done:
https://github.com/apache/tinkerpop/tree/TINKERPOP-2133
We still need the following issues addressed to get this merged to master:
1. checking into MultiMetaNeo4jTrait and modifying that as needed
1. checking for tests that validate these changes somehow? There are `TextP`
tests in the `HastTest` suite of `gremlin-test` but does anything in neo4j need
to be configured for the string matching to work?
1. needs upgrade docs.
> 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.3#76005)
