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