[jira] [Closed] (TINKERPOP-2800) ConnectiveP doesn't get evaluated right for list/set cardinality

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

     [ https://issues.apache.org/jira/browse/TINKERPOP-2800?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Stephen Mallette closed TINKERPOP-2800.
---------------------------------------
    Resolution: Not A Problem

starting to question if the first query was every right. conjunctives used that way should be taken to mean "for a single property" and therefore it would always be false if you did {{startingWith("2").and(startingWith("4"))}} as no one property can satisfy that. if you wanted the multi-property approach for {{and()}} you would have to do it in two {{has()}} like:
{{has('variant',startingWith("2").has('variant',startingWith("4"))}} - which i think is easier to reason about:

{code}
gremlin> g.V().has("variant", startingWith("2")).has('variant',startingWith("4")).valueMap()
==>[variant:[2wd,4wd]]
gremlin> g.V().has("variant", startingWith("2")).or(has('variant',startingWith("4")),has('variant',startingWith("6"))).valueMap()
==>[variant:[2wd,4wd]]
==>[variant:[2wd,6wd]]
{code}

> ConnectiveP doesn't get evaluated right for list/set cardinality
> ----------------------------------------------------------------
>
>                 Key: TINKERPOP-2800
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-2800
>             Project: TinkerPop
>          Issue Type: Bug
>          Components: process
>    Affects Versions: 3.5.4
>            Reporter: Stephen Mallette
>            Priority: Major
>
> Given:
> {code}
> g = TinkerGraph.open().traversal()
> g.addV().property(list,"variant","2wd").property(list,"variant","4wd").iterate()
> g.addV().property(list,"variant","2wd").property(list,"variant","6wd").iterate()
> g.addV().property(list,"variant","8wd").property(list,"variant","6wd").iterate()
> // 
> gremlin> g.V().has("variant",startingWith("2").and(startingWith("4"))).valueMap()
> ==>[variant:[2wd,4wd]]
> gremlin> g.V().or(has("variant",startingWith("2").and(startingWith("4"))),
> ......1>          has("variant",startingWith("2").and(startingWith("6")))).valueMap()
> gremlin> g.V().union(has("variant",startingWith("2").and(startingWith("4"))),
> ......1>             has("variant",startingWith("2").and(startingWith("6")))).valueMap()
> gremlin> g.V().has("variant",startingWith("2").and(startingWith("4"))).aggregate("x").fold().
> ......1>   V().has("variant",startingWith("2").and(startingWith("6"))).aggregate("x").fold().
> ......2>   cap("x").unfold().valueMap()
> ==>[variant:[2wd,4wd]]
> ==>[variant:[2wd,6wd]]
> {code}
> The {{HasContainer}} doesn't seem to get applied properly in the case of multi-properties. It sorta just treats properties as {{single}} in those cases.
> The consistency issue seems best resolved by following the latter cases and making it so that {{and()}} operates against a single property and prefer that multiproperties be resolved over multiple {{has()}}:
> {code}
> gremlin> g.V().has("variant", startingWith("2").and(startingWith("4"))).valueMap()
> gremlin> g.V().has("variant", startingWith("2")).has('variant',startingWith("4")).valueMap()
> ==>[variant:[2wd,4wd]]
> gremlin> g.V().has("variant", startingWith("2")).or(has('variant',startingWith("4")),has('variant',startingWith("6"))).valueMap()
> ==>[variant:[2wd,4wd]]
> ==>[variant:[2wd,6wd]]
> {code}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)