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