Gem Lamont

Results 63 comments of Gem Lamont

Fix double escaping graphml

Looks good, out of curiosity, where did you get the list for the invalid characters? are we sure that contains all of them? :)

apoc.convert.toTree() returns wrong result if there is a cycle from root of the path

Hello! Thank you for writing in :) There were some changes made in 5.18 to this procedure that might change the behaviour (adds rel id into the mix). I am...

Niyati

Hi! May I ask the purpose of this Pull Request? The README you added implies it is an extension, perhaps you mean to open a PR to your own fork?

apoc.refactor.cloneNodes() clones survive transaction rollback

Hi! thanks for reporting this, we will take a look :)

apoc.refactor.cloneNodes() clones survive transaction rollback

The fix will be available in 5.18 and 4.4.0.26 :)

Exception thrown when using `apoc.load.jdbc` about trying to close the connection before transaction is complete

Thanks for reporting, I've added a ticket for this for someone to investigate :)

add elementID in apoc.convert.toTree output as key-value pair (`_elementId`)

Hi! Thank you for writing in, I have added this suggestion to our work on migrating APOC to be more elementId friendly

Simple mysql select now(); Throws class java.time.LocalDateTime cannot be cast to class java.sql.Timestamp

Thank you for reporting this :) I have made a bug ticket for the team

Failed to invoke procedure `apoc.import.graphml`: Caused by: java.net.MalformedURLException: unknown protocol: d

Hi there! thanks for writing in :) You need to write file: at the start, otherwise the url tries to interpret the D: as the url protocol. Try: `CALL apoc.import.graphml("file:D:\code\my_repo\data\neo4j\all_members.gml",...

Failed to invoke procedure `apoc.import.graphml`: Caused by: java.net.MalformedURLException: unknown protocol: d

Hi! So it looks like the fix I supplied you works only for apoc.import.graphml, there does indeed seem to be a bug with `apoc.export.graphml.all` which is not working with Windows...