[GitHub] [age] jrgemignani commented on issue #321: SQL Command throws Error in cypher.sql

2022-10-18 Thread GitBox
jrgemignani commented on issue #321: URL: https://github.com/apache/age/issues/321#issuecomment-1282646770 Sorry, I should have noted the following before suggesting closing the issue - If those regression tests are supposed to throw errors **and** there isn't a comment stating that

[GitHub] [age] jrgemignani commented on issue #45: unique properties / indexing

2022-10-18 Thread GitBox
jrgemignani commented on issue #45: URL: https://github.com/apache/age/issues/45#issuecomment-1282657193 I will try to see if I can get others engaged that can help. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [age] TropicalPenguin commented on issue #330: Invalid Result Returned for OPTIONAL MATCH ... WITH ...

2022-10-18 Thread GitBox
TropicalPenguin commented on issue #330: URL: https://github.com/apache/age/issues/330#issuecomment-1282802614 Sorry @jrgemignani ; In trying to simplify the example (by using 0 as a placeholder for the actual ID), I hadn't realised I just made the actual issue harder to reproduce 😅.

[GitHub] [age] jrgemignani commented on issue #330: Invalid Result Returned for OPTIONAL MATCH ... WITH ...

2022-10-18 Thread GitBox
jrgemignani commented on issue #330: URL: https://github.com/apache/age/issues/330#issuecomment-1282936299 The following - `CREATE (a:A {foo:1})-[:incs]->(:C), (a)-[:incs]->(:C) RETURN a` creates 2 **distinct** paths joined at node **a**. So, 3 **distinct** nodes and 2 **disti

[GitHub] [age] TropicalPenguin commented on issue #330: Invalid Result Returned for OPTIONAL MATCH ... WITH ...

2022-10-18 Thread GitBox
TropicalPenguin commented on issue #330: URL: https://github.com/apache/age/issues/330#issuecomment-1282939163 > There isn't a path from a to c _and then back to a_, without reusing an edge. That's why it returns nothing. I've added emphasis to where this is wrong: the 'd' edge is a b

[GitHub] [age] TropicalPenguin commented on issue #330: Invalid Result Returned for OPTIONAL MATCH ... WITH ...

2022-10-18 Thread GitBox
TropicalPenguin commented on issue #330: URL: https://github.com/apache/age/issues/330#issuecomment-1282942228 In other words, it is expected to reuse the edge so as to be able to count the number of such 'dependencies' -- This is an automated message from the Apache Git Service. To respo

[GitHub] [age] jrgemignani commented on issue #330: Invalid Result Returned for OPTIONAL MATCH ... WITH ...

2022-10-18 Thread GitBox
jrgemignani commented on issue #330: URL: https://github.com/apache/age/issues/330#issuecomment-1282947056 I modified my answer above slightly to make it a little more clear. The bidirectional edge just means, any **available** edge connected to this node. It doesn't say to reuse an e

[GitHub] [age] TropicalPenguin commented on issue #330: Invalid Result Returned for OPTIONAL MATCH ... WITH ...

2022-10-18 Thread GitBox
TropicalPenguin commented on issue #330: URL: https://github.com/apache/age/issues/330#issuecomment-1282950694 While it is true that it doesn't _mandate_ reusing an edge, what I've come to expect from other implementations of Cypher is that it also isn't restricted to only following untrav

[GitHub] [age] jrgemignani commented on issue #330: Invalid Result Returned for OPTIONAL MATCH ... WITH ...

2022-10-18 Thread GitBox
jrgemignani commented on issue #330: URL: https://github.com/apache/age/issues/330#issuecomment-1282992051 It is not a limitation, it **depends** on the graph given and the "technical" **definition** of a path. But, just for a moment, think of the implications of allowing an edge to b

Re: [DISCUSS] Apache AGE for Postgres 12 - Version

2022-10-18 Thread Eya Badal
Thank you, everyone, for your great suggestions and feedback. Based on the votes and recommendations, I am closing this discussion; we will choose 1.1.0 prefix for this release. Thank you again, Eya On 2022/09/29 19:18:07 Eya Badal wrote: > Dear Everyone, > > We are planning on our first

[GitHub] [age-viewer] beinjamin commented on issue #45: Query Window Auto Rescale

2022-10-18 Thread GitBox
beinjamin commented on issue #45: URL: https://github.com/apache/age-viewer/issues/45#issuecomment-1283147369 hi -- 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. To unsubscr

[GitHub] [age-viewer] beinjamin commented on issue #21: Not compatible with AgensGraph?

2022-10-18 Thread GitBox
beinjamin commented on issue #21: URL: https://github.com/apache/age-viewer/issues/21#issuecomment-1283147589 hi -- 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. To unsubscr

[GitHub] [age] TropicalPenguin commented on issue #330: Invalid Result Returned for OPTIONAL MATCH ... WITH ...

2022-10-18 Thread GitBox
TropicalPenguin commented on issue #330: URL: https://github.com/apache/age/issues/330#issuecomment-1283474368 > Btw, as it is always possible that we got something incorrect or overlooked something, I ran your query on Neo4j to make sure that our implementation conforms with theirs and it