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