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

Consolidate feed_params and channel_index in config #1321

Closed
adamcfraser opened this issue Dec 3, 2015 · 1 comment
Closed

Consolidate feed_params and channel_index in config #1321

adamcfraser opened this issue Dec 3, 2015 · 1 comment
Assignees
Milestone

Comments

@adamcfraser
Copy link
Collaborator

Configuration of the channel index will be more straightforward if we move all configuration into the channel_index section of the config. Currently the feed_params are only relevant for distributed index writers.

We should move the feed_params.num_shards property under the channel_index section of the config, as it's only relevant to channel_index writers.

@tleyden
Copy link
Contributor

tleyden commented Dec 7, 2015

@sethrosetter just a reminder that you guys will need to update your configs once this change is made.

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