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

    https://github.com/apache/storm/pull/2433#discussion_r152696040
  
    --- Diff: 
storm-client/src/jvm/org/apache/storm/cluster/StormClusterStateImpl.java ---
    @@ -43,13 +44,15 @@
         private static Logger LOG = 
LoggerFactory.getLogger(StormClusterStateImpl.class);
     
         private IStateStorage stateStorage;
    +    private ILocalAssignmentsBackend backend;
    --- End diff --
    
    Let's make it clearer: `localAssignments` (or `assignments`) feels me more 
important, so maybe better to have it in field name.


---

Reply via email to