pzampino commented on a change in pull request #239: KNOX-2153 - CM discovery - Monitor Cloudera Manager URL: https://github.com/apache/knox/pull/239#discussion_r369022687
########## File path: gateway-discovery-cm/src/main/java/org/apache/knox/gateway/topology/discovery/cm/ClouderaManagerServiceDiscovery.java ########## @@ -63,6 +68,9 @@ static final String DEFAULT_USER_ALIAS = "cm.discovery.user"; static final String DEFAULT_PWD_ALIAS = "cm.discovery.password"; + private static final String GATEWAY_SERVICES_ACCESSOR_CLASS = "org.apache.knox.gateway.GatewayServer"; Review comment: This module doesn't depend on gateway-server...I think the services would have to be passed down to avoid this ---------------------------------------------------------------- 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] With regards, Apache Git Services
