Github user GJL commented on a diff in the pull request:
https://github.com/apache/flink/pull/5353#discussion_r165950588
--- Diff:
flink-docs/src/main/java/org/apache/flink/docs/rest/RestAPIDocGenerator.java ---
@@ -285,11 +287,22 @@ private static String createMessageHtmlEntry(Class<?>
messageClass, Class<?> emp
dispatcherGatewayRetriever = () -> null;
resourceManagerGatewayRetriever = () -> null;
+ transientBlobService =
NoOpTransientBlobService.INSTANCE;
metricQueryServiceRetriever = path -> null;
}
private DocumentingDispatcherRestEndpoint() {
- super(restConfig, dispatcherGatewayRetriever, config,
handlerConfig, resourceManagerGatewayRetriever, executor,
metricQueryServiceRetriever, NoOpElectionService.INSTANCE,
NoOpFatalErrorHandler.INSTANCE);
+ super(
+ restConfig,
+ dispatcherGatewayRetriever,
+ config,
+ handlerConfig,
+ resourceManagerGatewayRetriever,
+ transientBlobService,
--- End diff --
I don't think we need these `static final` variables. Can just use
`NoOpTransientBlobService.INSTANCE` as a constructor argument.
---
