[GitHub] storm pull request #2312: YSTORM-2725: Generic Resource Scheduling - initial...

[revans2]

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

    https://github.com/apache/storm/pull/2312#discussion_r138090258
  
    --- Diff: 
storm-client/src/jvm/org/apache/storm/topology/BaseConfigurationDeclarer.java 
---
    @@ -83,4 +83,29 @@ public T setCPULoad(Number amount) {
             }
             return (T) this;
         }
    +
    +    @SuppressWarnings("unchecked")
    +    @Override
    +    public T addResources(Map<String, Double> resources) {
    +        if(resources != null) {
    --- End diff --
    
    style: space after the `if` and before the `(`


---