Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TinkerPop 3.0 support #13

Open
wants to merge 26 commits into
base: tp3
Choose a base branch
from
Open

TinkerPop 3.0 support #13

wants to merge 26 commits into from

Conversation

amcp
Copy link
Contributor

@amcp amcp commented Mar 5, 2016

Upgrade use of traversal sources for TP3, other TP3 syntax
Use orient-gremlin until OrientDB is officially supported
Dependency version upgrades
Other small improvements

@sarovios
Copy link
Contributor

Do we have any performance benchmarks between TP API and Neo4j native API?

@sarovios
Copy link
Contributor

This is not related with this PR but I just find it out. I am not able to build the project, it seems that dynamoDBLocal is not on Maven Central as I get this: No versions available for com.amazonaws:DynamoDBLocal:jar:[1.10, 2.0.0). Please add the fix to README.md

@amcp
Copy link
Contributor Author

amcp commented Mar 13, 2016

I updated the README.md with some preliminary benchmark results. OrientDB is not officially at TP3 yet so I did not run these tests. I made some interesting findings especially with regards to how we were setting custom vertex ids in Titan. It turns out that our original implementation of custom vertex ids never worked. I found this out by enabling assertions (-ea) with the JVM and saw Titan core crap out in many places. I fixed the issues with setting custom ids in Titan, but I am not sure we stand to gain much from custom ids, so I will play around and see if using auto-ids improves performance.

@sarovios
Copy link
Contributor

Shall I close that and create a new PR when this is ready?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants