mas-chen commented on code in PR #24564:
URL: https://github.com/apache/flink/pull/24564#discussion_r1556248651


##########
docs/static/generated/rest_v1_dispatcher.yml:
##########
@@ -1089,6 +1089,37 @@ paths:
             application/json:
               schema:
                 $ref: '#/components/schemas/JobVertexBackPressureInfo'
+  /jobs/{jobid}/vertices/{vertexid}/coordinator-metrics:
+    get:
+      description: Provides access to job manager operator metrics

Review Comment:
   Sounds good. I'll add a short description. As for the method change 
suggestion, I'd rather keep the scope limited. It caused confusion in my 
implementation and it took some digging to figure out that the there is only 
one coordinator, at least in the Flink Public API. As it is an internal method, 
someone can easily change this down the road when the requirement surfaces



-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to