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

Dependency on AWS SDK #1094

Closed
shuzhang1989 opened this issue Apr 25, 2016 · 5 comments
Closed

Dependency on AWS SDK #1094

shuzhang1989 opened this issue Apr 25, 2016 · 5 comments

Comments

@shuzhang1989
Copy link
Contributor

Hi,

I am thinking to write an S3 environment for rocksdb, turned out we can leverage the AWS cpp sdk (https://github.com/aws/aws-sdk-cpp), but I wonder what's the best practice here to wind a 3rd party lib into rocks db?

@dhruba
Copy link
Contributor

dhruba commented Apr 26, 2016

This is a great initiative. Is the change all localized to a new rocksdb::Env?

We have similar integration of rocksdb + HDFS, see https://github.com/facebook/rocksdb/blob/master/util/env_hdfs.cc. If you can create a similar file for S3, that would be awesome!

@shuzhang1989
Copy link
Contributor Author

@dhruba yes, my assumption is that we can create another environment implementation separately.
The HDFS integration has the assumption that the user has HDFS library installed by themselves. Should we also push the responsibility of installing AWS sdk to users, or Rocksdb itself will integrate necessary aws libraries? Looks there is a third-party folder there.

@dhruba
Copy link
Contributor

dhruba commented Apr 26, 2016

My vote would be to make the user install the S3 libraries outside of rocksdb, but have a env_s3.cc inside of the rocksdb code. Basically, follow the same structure that we have done for HDFS

@shuzhang1989
Copy link
Contributor Author

@dhruba SGTM, thanks!

@gfosco
Copy link
Contributor

gfosco commented Jan 10, 2018

Closing this via automation due to lack of activity. If discussion is still needed here, please re-open or create a new/updated issue.

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

No branches or pull requests

3 participants