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

[filestore] Database binding for storing writes to the database in files #1053

Closed
wants to merge 0 commits into from

Conversation

c15yi
Copy link

@c15yi c15yi commented Oct 22, 2017

The idea for this binding is, to use the files for multiple benchmarks runs with different databases.
Using the exact same data should be good for consistency and makes the results more repeatable and reproducible.

Copy link
Collaborator

@busbey busbey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please place these into their own package.

What's an example of how one would use these files to load ycsb compatible data into a datastore? Since that's the listed purpose, it seems like we ought to have an example.

Given the stated purpose, why implement the run phase at all?

}

/**
* Reads the file with the name {@code table}_{@code key}.json.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This layout is going to create directories that folks have a hard time managing with even modest datasets. if we must keep with one-file-per-key, then I'd suggest making directories within the data directory. Something like 3 or 4 layers based on a prefix of the key.

}

private String getDatabaseFileName(String table, String key) {
return outputDirectory + table + "_" + key + ".json";
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How do I ensure that when I load these files into a datastore I'm doing it in the same order that YCSB generated them?

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