XComp commented on pull request #13547: URL: https://github.com/apache/flink/pull/13547#issuecomment-706126279
> Hi, @XComp > both solutions work for me, but I suggest not to remove the metric information from the `TaskManagerDetailsHandler` since it would bring break changes to the Rest API Thanks for your opinion. I agree with removing the metric information shouldn't be done right away. But we might consider it when thinking about updating the REST API in general having `v2` (as you mentioned as well with). There were a few things about the REST API that we could improve. Having a ticket proposing the cleanup of the metrics in the REST endpoint's response is just for documentation purposes for now. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
