Thanks for addressing some of my components @zentol. I think we should still change how the `ExecutorService` is instantiated. Instead of doing it in the constructor it should happen imo outside of the component. That way, we don't couple different components (e.g. which `ExecutorService` is used by the `WebMonitorEndpoint`). Moreover, it was already like this before and, thus, we would introduce a regression (in terms of architecture).
[ Full content available at: https://github.com/apache/flink/pull/6786 ] This message was relayed via gitbox.apache.org for [email protected]
