esecules commented on code in PR #9577:
URL: https://github.com/apache/nifi/pull/9577#discussion_r1935099915
##########
nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/FlowResource.java:
##########
@@ -555,11 +555,15 @@ public Response getBreadcrumbs(
public Response getFlowMetrics(
@Parameter(
description = "The producer for flow file metrics. Each
producer may have its own output format.",
- required = true
+ required = true,
+ schema = @Schema(allowableValues = {"prometheus", "json"})
)
@PathParam("producer") final String producer,
@Parameter(
- description = "Set of included metrics registries"
+ description = "Set of included metrics registries.
Duplicate the parameter to include multiple registries. " +
+ "All registries are included by default.",
+
+ schema = @Schema(allowableValues = {"NIFI", "JVM",
"BULLETIN", "CONNECTION", "CLUSTER"})
Review Comment:
Good to know. The YAML looks good from my side. Looking to see the v2 API
docs. I'll resolve this comment.
--
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]