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.

Sign up for GitHub

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

Jump to bottom

Make engine configurable #4140

Merged
merged 2 commits into from
Sep 17, 2015
Merged

Make engine configurable #4140

merged 2 commits into from
Sep 17, 2015

Conversation

otoolep
Copy link
Contributor

@otoolep otoolep commented Sep 17, 2015

Allows the engine to be selected. Default remains "bz1" so nothing actually changes.

Tested by setting bz1 and then b1 on the config, and seeing that a different engine type was passed to NewEngine on each occasion.

@jwilder
Copy link
Contributor

jwilder commented Sep 17, 2015

👍

otoolep added a commit that referenced this pull request Sep 17, 2015
@otoolep
Merge pull request #4140 from influxdb/engine_config
f73bc6f 
Make engine configurable
@otoolep otoolep merged commit f73bc6f into master Sep 17, 2015
@otoolep otoolep deleted the engine_config branch September 17, 2015 04:17
@otoolep
Copy link
Contributor Author

otoolep commented Sep 17, 2015

Cherry-picked to 0.9.4.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@otoolep @jwilder