zeroflag commented on code in PR #722:
URL: https://github.com/apache/knox/pull/722#discussion_r1094888769


##########
gateway-discovery-cm/src/main/java/org/apache/knox/gateway/topology/discovery/cm/ClouderaManagerServiceDiscoveryMessages.java:
##########
@@ -142,6 +142,12 @@ void 
failedToInstantiateJAASConfigurationFileImplementation(String implementatio
            text = "Terminating monitoring of {1} @ {0} for configuration 
changes because there are no referencing descriptors.")
   void stoppingConfigMonitoring(String discoverySource, String clusterName);
 
+  @Message(level = MessageLevel.WARN, text = "Missing property in previously 
saved service discovery configuration {0}")
+  void missingServiceDiscoveryConfigProperty(String propertyName);
+
+  @Message(level = MessageLevel.DEBUG, text = "There is no cluster 
configuration to check yet.")

Review Comment:
   There aren't any format string parameters in the message, but the method has 
2 arguments.



-- 
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: dev-unsubscr...@knox.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to