zentol commented on a change in pull request #16884:
URL: https://github.com/apache/flink/pull/16884#discussion_r692319782
##########
File path:
flink-runtime/src/main/java/org/apache/flink/runtime/io/disk/iomanager/IOManager.java
##########
@@ -54,6 +56,14 @@
protected IOManager(String[] tempDirs) {
this.fileChannelManager =
new
FileChannelManagerImpl(Preconditions.checkNotNull(tempDirs), DIR_NAME_PREFIX);
+ if (LOG.isInfoEnabled()) {
+ LOG.info(
+ "Created a new {} for spilling of task related data to
disk (joins, sorting, ...). Used directories: {}.",
+ FileChannelManager.class.getSimpleName(),
+ Arrays.stream(fileChannelManager.getPaths())
+ .map(File::getAbsolutePath)
+ .collect(Collectors.joining(", ")));
Review comment:
We already have a various logging statements that use multi-line log
messages without any user ringing alarm bells, so I'm not too worried at this
point.
--
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]
