Github user HeartSaVioR commented on a diff in the pull request:

    https://github.com/apache/storm/pull/1616#discussion_r73975193
  
    --- Diff: 
storm-core/src/jvm/org/apache/storm/executor/spout/SpoutExecutor.java ---
    @@ -79,8 +79,11 @@ public SpoutExecutor(final Map workerData, final 
List<Long> executorId, Map<Stri
             this.spoutThrottlingMetrics = new SpoutThrottlingMetrics();
         }
     
    -    @Override
         public void init(final Map<Integer, Task> idToTask) {
    +        while (!stormActive.get()) {
    --- End diff --
    
    I just follow previous executor implementation, and invoking init() to 
multiple times is not intended not only for while statement, but also rest of 
statements.


---
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.
---

Reply via email to