PatrickRen commented on a change in pull request #16951:
URL: https://github.com/apache/flink/pull/16951#discussion_r701591943



##########
File path: 
flink-runtime/src/main/java/org/apache/flink/runtime/rest/RestClient.java
##########
@@ -138,7 +138,7 @@ public RestClient(Configuration configuration, Executor 
executor)
                 final OutboundChannelHandlerFactory factory = factories.next();
                 if (factory != null) {
                     outboundChannelHandlerFactories.add(factory);
-                    LOG.info("Loaded channel outbound factory: {}", factory);
+                    LOG.debug("Loaded channel outbound factory: {}", factory);

Review comment:
       This INFO log is too much, and not quite meaningful. I assume this is 
set by mistake for debugging: 
   ```
   180248 [main] INFO  org.apache.flink.runtime.rest.RestClient [] - Loaded 
channel outbound factory: 
org.apache.flink.runtime.rest.Prio0OutboundChannelHandlerFactory@4b6e1c0
   180248 [main] INFO  org.apache.flink.runtime.rest.RestClient [] - Loaded 
channel outbound factory: 
org.apache.flink.runtime.rest.Prio1OutboundChannelHandlerFactory@561b61ed
   180254 [main] INFO  org.apache.flink.runtime.rest.RestClient [] - Loaded 
channel outbound factory: 
org.apache.flink.runtime.rest.Prio0OutboundChannelHandlerFactory@15400fff
   180254 [main] INFO  org.apache.flink.runtime.rest.RestClient [] - Loaded 
channel outbound factory: 
org.apache.flink.runtime.rest.Prio1OutboundChannelHandlerFactory@18d910b3
   180299 [main] INFO  org.apache.flink.runtime.rest.RestClient [] - Loaded 
channel outbound factory: 
org.apache.flink.runtime.rest.Prio0OutboundChannelHandlerFactory@77774571
   180299 [main] INFO  org.apache.flink.runtime.rest.RestClient [] - Loaded 
channel outbound factory: 
org.apache.flink.runtime.rest.Prio1OutboundChannelHandlerFactory@277b8fa4
   ```




-- 
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