davidradl commented on code in PR #23465:
URL: https://github.com/apache/flink/pull/23465#discussion_r1337245273
##########
flink-runtime/src/main/java/org/apache/flink/runtime/rest/messages/SubtasksTimesInfo.java:
##########
@@ -132,13 +138,14 @@ public boolean equals(Object o) {
SubtaskTimeInfo that = (SubtaskTimeInfo) o;
return subtask == that.subtask
&& Objects.equals(host, that.host)
Review Comment:
will host be `(unassigned)` if not set like the deprecated location?
I assume that as the old form did not have the port, the old form and new
form cannot ever be equivalent.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
