Github user srdo commented on the issue:
https://github.com/apache/storm/pull/2297
@liu-zhaokun No, not as it is currently. I think Curator already handles
failing over to another Zookeeper server for us, so if the user passes a zkStr
with multiple servers here
https://github.com/apache/storm/blob/master/external/storm-kafka/src/jvm/org/apache/storm/kafka/DynamicBrokersReader.java#L61
and sets the retry policy at
https://github.com/apache/storm/blob/master/external/storm-kafka/src/jvm/org/apache/storm/kafka/DynamicBrokersReader.java#L64
to a few attempts, then I don't see why we need to add another layer of
retries on top of that?
Plus as @HeartSaVioR points out, this doesn't compile.
---
