You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@storm.apache.org by krichter722 <gi...@git.apache.org> on 2018/03/19 20:04:34 UTC

[GitHub] storm issue #460: [STORM-3000] added missing @Override annotations

Github user krichter722 commented on the issue:

    https://github.com/apache/storm/pull/460
  
    @d2r I rebased the branch, removed changes in classes generated by the Thrift compiler and included all subprojects. Since checkstyle is now in place I reviewed all `maxAllowedViolations` in POMs in order to avoid future missing `@Override`s.
    
    I can reduce the `maxAllowedViolations` values is a separate PR, but I think it makes sense to include them here.
    
    Please reopen this PR (I can't do it) can trigger Travis.


---