azagrebin commented on a change in pull request #8680: [FLINK-12706] Introduce 
ShuffleService interface and its configuration
URL: https://github.com/apache/flink/pull/8680#discussion_r294663770
 
 

 ##########
 File path: 
flink-runtime/src/main/java/org/apache/flink/runtime/io/network/partition/consumer/SingleInputGateFactory.java
 ##########
 @@ -50,10 +50,11 @@
  * Factory for {@link SingleInputGate} to use in {@link 
NettyShuffleEnvironment}.
  */
 public class SingleInputGateFactory {
+       @SuppressWarnings("LoggerInitializedWithForeignClass")
        private static final Logger LOG = 
LoggerFactory.getLogger(SingleInputGate.class);
 
 Review comment:
   My idea was to preserve the structure of the previous logging messages 
because the factory was a result of refactoring. I can change this if it is not 
important.

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

Reply via email to