[
https://issues.apache.org/jira/browse/FLINK-29872?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17628684#comment-17628684
]
Gyula Fora commented on FLINK-29872:
------------------------------------
Our goal is not to generate new clients if they are already available in Flink.
Best would be to simply use it as we already use many other utilities.
You actually provided an excellent argument why we should make the RestClient
backward compatible. As you said a simple generated client based on the latest
spec would probably work. In that case I believe we should improve the
RestClient so that it comes with this nice property also. Why would we design a
non backward compatible RestClient if the spec itself is backward compatible?
> JobDetailsInfo Rest endpoint breaking change in 1.16
> ----------------------------------------------------
>
> Key: FLINK-29872
> URL: https://issues.apache.org/jira/browse/FLINK-29872
> Project: Flink
> Issue Type: Bug
> Components: Runtime / REST
> Affects Versions: 1.16.0
> Reporter: Gyula Fora
> Priority: Major
>
> Flink 1.16 introduces a breaking change to the JobDetailsInfo endpoints by
> adding 3 required fields to:
> [https://github.com/apache/flink/blob/master/flink-runtime/src/main/java/org/apache/flink/runtime/rest/messages/job/metrics/IOMetricsInfo.java]
> accumulatedBackpressured, accumulatedIdle, accumulatedBusy fields should not
> be primitive (which makes them required) but instead nullable.
> This would allow the 1.16 restclient to read the jobdetails of previous
> cluster versions
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
