Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2839#discussion_r218564458
--- Diff:
storm-server/src/main/java/org/apache/storm/localizer/TimePortAndAssignment.java
---
@@ -48,4 +48,29 @@ public LocalAssignment getAssignment() {
public void complete() {
stopTiming();
}
+
+ @Override
+ public String toString() {
+ return "TimePortAndAssignment{" +
getAssignment().get_topology_id() + " on " + getPort() + "}";
+ }
+
+ /**
+ * All implementations of PortAndAssignment should implement the same
hashCode() method
+ */
+ @Override
+ public int hashCode() {
+ return (17 * getPort()) + getAssignment().hashCode();
+ }
+
+ /**
+ * All implementations of PortAndAssignment should implement the same
equals() method
--- End diff --
Checkstyle: missing period here too.
---
