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

    https://github.com/apache/storm/pull/2839#discussion_r218564194
  
    --- Diff: 
storm-server/src/main/java/org/apache/storm/localizer/PortAndAssignmentImpl.java
 ---
    @@ -26,13 +26,16 @@ public PortAndAssignmentImpl(int port, LocalAssignment 
assignment) {
             this.assignment = assignment;
         }
     
    +    /**
    +     * All implementations of PortAndAssignment should implement the same 
equals() method
    --- End diff --
    
    checkstyle: missing period at the end of this.


---

Reply via email to