Re: [I] MERGE incorrectly creates multiple vertices [age]

2024-04-02 Thread via GitHub
mingfang commented on issue #1691: URL: https://github.com/apache/age/issues/1691#issuecomment-2033504774 @jrgemignani confirm fixed. Thank you! -- 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

Re: [I] MERGE incorrectly creates multiple vertices [age]

2024-04-02 Thread via GitHub
mingfang closed issue #1691: MERGE incorrectly creates multiple vertices URL: https://github.com/apache/age/issues/1691 -- 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

Re: [I] MERGE incorrectly creates multiple vertices [age]

2024-04-02 Thread via GitHub
jrgemignani commented on issue #1691: URL: https://github.com/apache/age/issues/1691#issuecomment-2033277137 @mingfang This should be fixed in the master branch now. If you could verify it, and close the ticket provided it resolves the issue, that would be great. -- This is an automated

Re: [I] MERGE incorrectly creates multiple vertices [age]

2024-04-01 Thread via GitHub
mingfang commented on issue #1691: URL: https://github.com/apache/age/issues/1691#issuecomment-2030778828 @jrgemignani Amazing. Thanks again! -- 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

Re: [I] MERGE incorrectly creates multiple vertices [age]

2024-04-01 Thread via GitHub
jrgemignani commented on issue #1691: URL: https://github.com/apache/age/issues/1691#issuecomment-2030768325 @mingfang PR #1718 should address this issue once it is reviewed and merged. -- This is an automated message from the Apache Git Service. To respond to the message, please log on

Re: [I] MERGE incorrectly creates multiple vertices [age]

2024-03-30 Thread via GitHub
jrgemignani commented on issue #1691: URL: https://github.com/apache/age/issues/1691#issuecomment-2028143137 @mingfang Update: I currently have a fix for this and am reviewing it, creating test cases, and cleaning up the code. If all goes well, a PR should be available Monday or Tuesday.

Re: [I] MERGE incorrectly creates multiple vertices [age]

2024-03-21 Thread via GitHub
mingfang commented on issue #1691: URL: https://github.com/apache/age/issues/1691#issuecomment-2013202968 @jrgemignani Thanks for the update. I appreciate all your hard work. Thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [I] MERGE incorrectly creates multiple vertices [age]

2024-03-21 Thread via GitHub
jrgemignani commented on issue #1691: URL: https://github.com/apache/age/issues/1691#issuecomment-2013170834 @mingfang The particular cause for this issue is hard to easily describe, especially without getting very technical. Suffice to say, that MERGE (within the execution of a MERGE

Re: [I] MERGE incorrectly creates multiple vertices [age]

2024-03-20 Thread via GitHub
jrgemignani commented on issue #1691: URL: https://github.com/apache/age/issues/1691#issuecomment-2010464674 @mingfang This is an interesting one. -- 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

Re: [I] MERGE incorrectly creates multiple vertices [age]

2024-03-18 Thread via GitHub
jrgemignani commented on issue #1691: URL: https://github.com/apache/age/issues/1691#issuecomment-2004738633 Yeah, the more that I look at it, the more it appears to fall there. Transaction bugs, though, can be a pain to fix. Especially, for merge. -- This is an automated message from

Re: [I] MERGE incorrectly creates multiple vertices [age]

2024-03-18 Thread via GitHub
mingfang commented on issue #1691: URL: https://github.com/apache/age/issues/1691#issuecomment-2004732935 Also this is only a problem when multiple MERGE are called in the same query. e.g. this works as expected, only one vertex is created ``` SELECT * FROM cypher('playground', $$

Re: [I] MERGE incorrectly creates multiple vertices [age]

2024-03-18 Thread via GitHub
mingfang commented on issue #1691: URL: https://github.com/apache/age/issues/1691#issuecomment-2004705333 I'm sorry but I don't think that's correct. I've tested this in neo4j and it only results in one vertex created, not two like in AGE. Here is the neo4j doc that speaks directly to

Re: [I] MERGE incorrectly creates multiple vertices [age]

2024-03-18 Thread via GitHub
jrgemignani commented on issue #1691: URL: https://github.com/apache/age/issues/1691#issuecomment-2004696804 @mingfang Yes. I was just stating that this is also the case for edges. Vertices can have duplicate properties and edges can have duplicate properties. Properties are not unique.

Re: [I] MERGE incorrectly creates multiple vertices [age]

2024-03-18 Thread via GitHub
mingfang commented on issue #1691: URL: https://github.com/apache/age/issues/1691#issuecomment-2004655129 @jrgemignani please note in my test case the problem is that there MERGE created two vertices with the same name, and not one vertex and one edge. -- This is an automated message

[I] MERGE incorrectly creates multiple vertices [age]

2024-03-16 Thread via GitHub
mingfang opened a new issue, #1691: URL: https://github.com/apache/age/issues/1691 ```sql SELECT * FROM cypher('playground', $$ unwind ["foo", "foo"] as each merge (v:TEST {name: each}) return v $$) AS (v agtype) ``` returns ```json [ { "v": {