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

Adding Support for GridDB #1018

Closed
wants to merge 3 commits into from

Conversation

rbrasga
Copy link

@rbrasga rbrasga commented Sep 7, 2017

Hello,

I recently started using GridDB with YCSB, and I was surprised to see it wasn't part of the primary YCSB git repo.

I have it compiled and running. I realize I may need to make the README more clean and clear, so let me know what needs to be done to get this added to the official YCSB repo.

Thanks,
-Remi

@busbey
Copy link
Collaborator

busbey commented Sep 7, 2017

Thanks for your interest!

Is the effort here to directly port the griddb binding from griddb/griddb_ycsb? Or a reimplementation?

@rbrasga
Copy link
Author

rbrasga commented Sep 7, 2017

Is the effort here to directly port the griddb binding from griddb/griddb_ycsb? Or a reimplementation?

The effort is to directly port the griddb binding. If you are already working with them, then you could ignore this pull request.

I will look into the error and get it resolved.

@busbey
Copy link
Collaborator

busbey commented Sep 9, 2017

Have you approached the griddb folks about merging in here? I'm hesitant to fork their binding if they're still maintaining it.

@rbrasga
Copy link
Author

rbrasga commented Sep 9, 2017 via email

@busbey
Copy link
Collaborator

busbey commented Sep 9, 2017

if you rebase to the current master branch the travis failures will probably be addressed; we just pushed some fixes.

@busbey
Copy link
Collaborator

busbey commented Apr 26, 2019

obsoleted by #1258

@busbey busbey closed this Apr 26, 2019
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