exceptionfactory commented on code in PR #9577:
URL: https://github.com/apache/nifi/pull/9577#discussion_r1934902184
##########
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:
That HTML rendering is from version 1, NiFi 2 uses a simpler HTML template
that should include the allowable values. As long as the enumerated values show
up in the YAML, we can address the HTML rendering separately.
--
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]
