Github user harshach commented on the pull request:
https://github.com/apache/storm/pull/354#issuecomment-73448524
@Parth-Brahmbhatt
1. I think I misunderstood the config here if your initial intent is to
keep cluster wide and keep a storm.yaml config for cluster wide replication and
have min replication I think what you have is works fine. I thought this
config can be overridden by the topology users and hence my comment. Please
ignore as it doesn't look like your intent.
2. same as above
3. sounds good
5. Agree with you
7. This is my bad and you are right , the current code you have works fine.
8. Thanks for the details , ok now I understood it bit better :)
9. Sounds good.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
