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
Add command line options for configuration #7
Comments
Merged
ntietz
added a commit
that referenced
this issue
Oct 14, 2022
ntietz
added a commit
that referenced
this issue
Oct 17, 2022
This creates preliminary command-line configuration options and plumbs it to pass these through to their consumers. The initial options are for configuring number of worker threads, size of a queue, bind address, and storage base path, and more will come as needed. These allow easy tuning for benchmarks and optimizations. More should be added based on #7 , but these are what's available for what's implemented right now. In the future, as we add features, the corresponding flags should be added.
ntietz
added a commit
that referenced
this issue
Nov 5, 2023
ntietz
added a commit
that referenced
this issue
Nov 5, 2023
This creates preliminary command-line configuration options and plumbs it to pass these through to their consumers. The initial options are for configuring number of worker threads, size of a queue, bind address, and storage base path, and more will come as needed. These allow easy tuning for benchmarks and optimizations. More should be added based on #7 , but these are what's available for what's implemented right now. In the future, as we add features, the corresponding flags should be added.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
There's no configuration right now, but with the addition of durability ( #5 ) there are tangible things we need to be able to toggle on and off for benchmarks. This should also allow configuration of other properties for nice experimentation. Included in scope is the creation of a config struct to pass around (probably in the context struct).
Some things we probably want as options:
And probably more!
The text was updated successfully, but these errors were encountered: