zhuzhurk commented on code in PR #27876:
URL: https://github.com/apache/flink/pull/27876#discussion_r3044344452
##########
flink-runtime/src/main/java/org/apache/flink/runtime/rest/messages/checkpoints/SubtaskCheckpointStatistics.java:
##########
@@ -156,6 +158,9 @@ public static final class
CompletedSubtaskCheckpointStatistics
@JsonProperty(FIELD_NAME_ABORTED)
private final boolean aborted;
+ @JsonProperty(value = FIELD_NAME_IP_ADDRESS, required = false)
+ private final String ip;
Review Comment:
How about having the web UI fetch subtask information via the existing REST
API [1] and using the endpoint field directly? This approach would allow us to
avoid modifying the REST API or any backend code.
Since the endpoint field contains both the hostname and port, it arguably
provides more useful context than just the IP address.
[1]
https://nightlies.apache.org/flink/flink-docs-stable/docs/ops/rest_api/#jobs-jobid-vertices-vertexid
--
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]
