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

Implemented OrientDB client #76

Closed
wants to merge 5 commits into from
Closed

Implemented OrientDB client #76

wants to merge 5 commits into from

Conversation

lvca
Copy link
Contributor

@lvca lvca commented May 12, 2012

I've just implemented the client for OrientDB document graph NoSQL dbms (http://www.orientdb.org).

This implementation uses the "local" protocol that means the database is embedded.

The code it's very simple (few lines) and I've not played too much to improve performance.

Do you have a comparison against other clients with the same HW?

@ghost
Copy link

ghost commented May 14, 2012

Hi Luca,

Thank you for the patch! Could you add an DATABASES entry in bin/ycsb for orientdb? It should look something like:

"orientdb"    : "com.yahoo.ycsb.db.OrientDBClient",

Thanks!
--Michi

@ghost
Copy link

ghost commented May 14, 2012

Unfortunately the build is failing. It looks like it cannot find orientdb-core.jar.

[ERROR] Failed to execute goal on project orientdb-binding: Could not resolve dependencies for project com.yahoo.ycsb:orientdb-binding:jar:0.1.4: Could not find artifact com.orientechnologies:orientdb-core:jar:1.0-SNAPSHOT -> [Help 1]

Could you take a look?

Thanks!
--Michi

@lvca
Copy link
Contributor Author

lvca commented May 14, 2012

Hi, fixed:

  • ycsb file by adding orientdb
  • included snapshot repository in pom.xml
  • updated readme.md
    Lvc@

ghost pushed a commit that referenced this pull request Aug 14, 2012
ghost pushed a commit that referenced this pull request Aug 14, 2012
@ghost
Copy link

ghost commented Aug 14, 2012

Finally merged the patch. Sorry it took me so long, and thanks for the patch, Luca!

--Michi

@ghost ghost closed this Aug 14, 2012
jbohman pushed a commit to jbohman/YCSB that referenced this pull request Sep 11, 2012
jbohman pushed a commit to jbohman/YCSB that referenced this pull request Sep 11, 2012
joey pushed a commit to joey/YCSB that referenced this pull request Feb 7, 2013
joey pushed a commit to joey/YCSB that referenced this pull request Feb 7, 2013
jaricftw pushed a commit to jaricftw/YCSB that referenced this pull request Jul 19, 2016
jaricftw pushed a commit to jaricftw/YCSB that referenced this pull request Jul 19, 2016
jaricftw pushed a commit to jaricftw/YCSB that referenced this pull request Jul 19, 2016
jaricftw pushed a commit to jaricftw/YCSB that referenced this pull request Jul 19, 2016
This pull request was closed.
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.

1 participant