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


##########
gateway-spi/src/main/java/org/apache/knox/gateway/SpiGatewayMessages.java:
##########
@@ -91,10 +91,10 @@ public interface SpiGatewayMessages {
             text = "The dispatch to {0} was disallowed because it fails the 
dispatch whitelist validation. See documentation for dispatch whitelisting." )
   void dispatchDisallowed(String uri);
 
-  @Message( level = MessageLevel.DEBUG, text = "HTTP client connection timeout 
is set to {0} for {1}" )
+  @Message( level = MessageLevel.INFO, text = "HTTP client connection timeout 
is set to {0} for {1}" )
   void setHttpClientConnectionTimeout(int connectionTimeout, String 
serviceRole);
 
-  @Message( level = MessageLevel.DEBUG, text = "HTTP client socket timeout is 
set to {0} for {1}" )
+  @Message( level = MessageLevel.INFO, text = "HTTP client socket timeout is 
set to {0} for {1}" )

Review Comment:
   Looks good. Maybe adding the unit of measurements (msec I guess) for both 
the connection and socket timeouts would useful.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to