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