Matteo Kamm

Results 18 comments of Matteo Kamm

I did some more testing and this seems to only happen if `lTrim` appears in the boolean expression.

This seems to be fixed now in newer versions (confirmed on 4.4.10 enterprise).

A similar thing happens with the query: ``` CREATE ({property: right("OD", 3708444074355087135)}) ``` which throws ``` Exception in thread "main" java.lang.IndexOutOfBoundsException: negative length at org.neo4j.cypher.operations.CypherFunctions.right(CypherFunctions.java:882) at org.neo4j.cypher.internal.runtime.interpreted.commands.expressions.RightFunction.compute(StringFunctions.scala:186) at org.neo4j.cypher.internal.runtime.interpreted.commands.expressions.NullInNullOutExpression.apply(NullInNullOutExpression.scala:32) at...

Yes you are correct, this would throw an exception in Neo4J. But the following example works on both graph databases: ``` CREATE (:L {p:0.0/0.0}) CREATE INDEX FOR (n:L) ON (n.p)...

Yes you are right, since we compare `10 = NaN` it should be `false`. The `COUNT(n)` part is actually correct. In particular, the following queries seem to return incorrect results:...

A similar situation arises from these queries: ``` CREATE (:L {p:"\r"}) CREATE INDEX FOR (n:L) ON (n.p) MATCH (n:L) RETURN n.p = "\r" MATCH (n:L) WHERE n.p = "\r" RETURN...

@gkorland Ah I see, that would be problematic. So what do you think is the best solution here? Leave it as is but maybe state in the documentation that this...

It seems like Neo4J implemented it according to the [IEEE 754](https://en.wikipedia.org/wiki/IEEE_754) floating-point standard

One more example that seems to throw the same exception: ```java PropertiesConfiguration conf = ConfigurationUtil.loadPropertiesConfig("conf/test.properties"); JanusGraph graph = JanusGraphFactory.open(conf); JanusGraphManagement m = graph.openManagement(); PropertyKey key = m.makePropertyKey("prop").dataType(Date.class).make(); m.buildIndex("propIndex", Vertex.class).addKey(key).buildMixedIndex("search"); m.commit();...

@evan2645 Thanks for getting back to me. Yes, my intention was to implement this for the SPIRE server first. I've also thought about a use-case for a similar plugin for...