search

ArangoDB-Community / arangodb-tinkerpop-provider

An implementation of the Tinkerpop OLTP Provider for ArangoDB
Apache License 2.0
83 stars 16 forks source link

Support Integer and UUID ids #37

Open arcanefoam opened 5 years ago

arcanefoam commented 5 years ago

Supporting this types of ids works for serialization (storing in the DB), however when desrialiazing (reading the DB) it does not. The reason is that Integers and UUIDs are stored as strings in the DB and deserialized as such. This causes a problem if the user (i.e. tinkerpop engine/console) is expecting an actual Integer and UUID. We could use the same workaround as for node attributes by storing the expected id class in the DB and then using this information for deserialization.

a-marcel commented 5 years ago

Hello, if someone just copy paste your Readme code the part with the "element IDs" will not work. The reason is that the ArangoDBConfigurationBuilder prefix all collections with "graph_". So your code example will not work if you just use "person/1" you need to query "graph_person/1".

It's not a big deal but maybe it confuses arangodb graph newbies (like me) :)

Marcel

arcanefoam commented 5 years ago

Thanks for the update. I will take a look and update the readme (or maybe the code).