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

    https://github.com/apache/storm/pull/2551#discussion_r166749574
  
    --- Diff: 
storm-client/src/jvm/org/apache/storm/grouping/LoadAwareShuffleGrouping.java ---
    @@ -44,7 +44,7 @@
     import org.slf4j.LoggerFactory;
     
     public class LoadAwareShuffleGrouping implements 
LoadAwareCustomStreamGrouping, Serializable {
    -    static final int CAPACITY = 1000;
    +    private int CAPACITY;
    --- End diff --
    
    If we no longer use it as constant, the name should be changed to lower 
case.


---

Reply via email to