apucher commented on a change in pull request #6822:
URL: https://github.com/apache/incubator-pinot/pull/6822#discussion_r617099123
##########
File path: pinot-minion/src/main/java/org/apache/pinot/minion/MinionStarter.java
##########
@@ -68,6 +71,8 @@
private final HelixManager _helixManager;
private final TaskExecutorFactoryRegistry _taskExecutorFactoryRegistry;
private final EventObserverFactoryRegistry _eventObserverFactoryRegistry;
+ private MinionAdminApiApplication _minionAdminApplication;
+ private final List<ListenerConfig> _listenerConfigs;
public MinionStarter(String helixClusterName, String zkAddress,
PinotConfiguration config)
throws Exception {
Review comment:
You'll need one additional call to `installDefaultSSLFactory` here. see
ControllerStarter `setUpPinotController()`
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
