zentol commented on code in PR #22355:
URL: https://github.com/apache/flink/pull/22355#discussion_r1159697530
##########
flink-runtime/src/main/java/org/apache/flink/runtime/rest/messages/job/JobDetailsInfo.java:
##########
@@ -259,6 +261,7 @@ public String getJsonPlan() {
// ---------------------------------------------------
/** Detailed information about a job vertex. */
+ @Schema(name = "JobDetailsVertexInfo")
public static final class JobVertexDetailsInfo {
Review Comment:
There are of course advantages to renaming the class, like it being easier
to track down the internal representation of something in the API. But I'm not
sure if that's worth the hassle of restricting us to unique class names
everywhere.
--
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]