[GitHub] storm issue #2657: STORM-3048 : A Potential NPE

[revans2 <gi...@git.apache.org>]

Github user revans2 commented on the issue:

    https://github.com/apache/storm/pull/2657
  
    I am getting a failure now of
    
    ```
    [ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:2.17:check (validate) on project storm-server: You have 792 Checkstyle violations. The maximum number of allowed violations is 783. -> [Help 1]
    ```
    
    Specifically
    
    ```
    /Users/evans/src/apache-commit/storm-server/src/main/java/org/apache/storm/scheduler/resource/strategies/scheduling/ConstraintSolverStrategy.java
    	101: WhitespaceAround: 'assert' is not followed by whitespace. Empty blocks may only be represented as {} when not part of a multi-block statement (4.1.3)
    	101: WhitespaceAround: '!=' is not preceded with whitespace.
    	101: WhitespaceAround: '!=' is not followed by whitespace. Empty blocks may only be represented as {} when not part of a multi-block statement (4.1.3)
    	139: WhitespaceAround: 'assert' is not followed by whitespace. Empty blocks may only be represented as {} when not part of a multi-block statement (4.1.3)
    	139: WhitespaceAround: '!=' is not preceded with whitespace.
    	139: WhitespaceAround: '!=' is not followed by whitespace. Empty blocks may only be represented as {} when not part of a multi-block statement (4.1.3)
    	178: WhitespaceAround: 'assert' is not followed by whitespace. Empty blocks may only be represented as {} when not part of a multi-block statement (4.1.3)
    	178: WhitespaceAround: '!=' is not preceded with whitespace.
    	178: WhitespaceAround: '!=' is not followed by whitespace. Empty blocks may only be represented as {} when not part of a multi-block statement (4.1.3)
    ```


---