neo4j-rstats / neo4r

A Modern and Flexible Neo4J Driver
https://neo4j-rstats.github.io/user-guide/
Other
106 stars 30 forks source link

neo4r not working with latest Neo4j Version 4.0 #69

Open MarianTie opened 4 years ago

MarianTie commented 4 years ago

When trying to ping the latest neo4j database version 4.0, i receive this message:

> con$ping()
[1] 404

With the previous version 3.5.14 everything works fine.

It would be nice to update neo4r so that the connetion is also possible with the newest neo4j version.

davidmeza1 commented 4 years ago

Same issue. Any thoughts.

fBedecarrats commented 4 years ago

Well, for now you might just stick with the 3.5 version of Neo4J and it works.

ColinFay commented 4 years ago

I'd be happy to review a PR that update this

markjhill commented 4 years ago

Just adding my name to the list of people having an issue with this.

ColinFay commented 4 years ago

Hey folks,

Can you confirm that only the ping() method fails, not the rest of the package?

Cheers, Colin

markjhill commented 4 years ago

Hey folks,

Can you confirm that only the ping() method fails, not the rest of the package?

Cheers, Colin

Not super helpful but - so far - yes. But my testing hasn't been entirely thorough (I'm in the DB creation stages).

MarianTie commented 4 years ago

Hey folks,

Can you confirm that only the ping() method fails, not the rest of the package?

Cheers, Colin

Some weeks ago it didnt work. Now I can import the data with the neo4j version 4.0 into R too! The ping(), get_labels(), get_relationships() and get_version() methods fail. I dont have constraints or indexes so i cant check these functions. ( i get for the latter two Null data.table (0 rows and 0 cols)) Cant say something to the reset functions.

Hope it helps!

Never said, thank you a lot for this nice package! Helps a lot!

ColinFay commented 4 years ago

Thanks @MarianTie

I suspect there has been some changes in the REST api endpoint names, and the fact that some methods don't work anymore seem to confirm that, and confirms that it's not the connection itself which is broken, thanks for checking.

I'll should have some time to check on this by the end of this month.

(But if anybody wants to tackle that in the meantime, I would be happy to help pointing to where to look and review a PR)

Colin

ColinFay commented 4 years ago

Seems like the endpoints powering these methods have been removed :

https://neo4j.com/docs/rest-docs/current/

markcditzel commented 4 years ago

Dear All, Really wanted to have a try using Neo4r today. Spent best part of the day trying to work out was was wrong before finding this post! If anyone else was following Colin's excellent guide - just don't worry about the 404 errors with con%ping and just proceed to Chapter 4 - where the call_neo4j() works.