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

Design of external storage + RocksDB interface #81

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bashimao
Copy link
Collaborator

@bashimao bashimao commented Jan 17, 2023

Revised the MR. This now only contains the external storage interface + rocks db template implementation + tests.

@bashimao bashimao requested a review from rhdong January 17, 2023 12:45
@github-actions
Copy link

@bashimao bashimao self-assigned this Feb 2, 2023
@bashimao bashimao added the enhancement New feature or request label Feb 2, 2023
include/merlin_hashtable.cuh Outdated Show resolved Hide resolved
@bashimao bashimao force-pushed the fea-gds-interface branch 2 times, most recently from eb42b56 to b7907b5 Compare February 9, 2023 02:37
@bashimao bashimao force-pushed the fea-gds-interface branch from b7907b5 to b5bce87 Compare June 8, 2023 18:14
@bashimao bashimao changed the title Design of GDS Interface Design of external storage + RocksDB interface Jun 8, 2023
@bashimao
Copy link
Collaborator Author

bashimao commented Jun 8, 2023

@rhdong I changed the PR. See this as part 1 of the GDS interface .

The external storage interface as specified now, will be used by the feeding process.

@bashimao
Copy link
Collaborator Author

bashimao commented Jun 8, 2023

RocksDB is downgraded to version 6.29 because that is the last version that doesn't yet require C++17.

@bashimao bashimao force-pushed the fea-gds-interface branch from b5bce87 to 7be9c65 Compare June 9, 2023 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants