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

Too many config files #5

Open
estezz opened this issue Dec 12, 2014 · 4 comments
Open

Too many config files #5

estezz opened this issue Dec 12, 2014 · 4 comments

Comments

@estezz
Copy link

estezz commented Dec 12, 2014

I have 6 topics each with 3 partitions. that makes 18 config files and processes. It there a way to reduce this?

@reachkrishnaraj
Copy link
Owner

@estezz : Thanks, Yes, thats a good point which I am already thinking to implement. Should not take more than 1 hr of time. I will work on it and let you know. There are 2 ways to do this.1

  1. Keep the consumer job single threaded job which will be tied to each partition. And have 1 common property file for all the consumer instances(without the partition parameter) and pass the topic as command line parameter while starting the job.

  2. Multiple thread the consumer job, 1 thread for each topic. And all the topics can be put in a comma separated single property.

@estezz
Copy link
Author

estezz commented Dec 12, 2014

I think #2 sounds best. It will make monitoring the consumer easier. If you want help let me know.
Secor https://github.com/pinterest/secor has a nice configuration with wildcarding

@reachkrishnaraj
Copy link
Owner

@estezz : hmm, but we need to think about this scenario where,say only 1 consumer instance only has problem. In that case, we will need to bring all our other instances/threads down causing lag to in the indexing. This will be costly for index heavy consumers. I am think it will be good to implement both and provide an option in the config for folks to choose between option 1 and 2.

@estezz
Copy link
Author

estezz commented Dec 12, 2014

Both options would be awesome.

reachkrishnaraj pushed a commit that referenced this issue Sep 21, 2015
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

No branches or pull requests

2 participants