Github user GJL commented on a diff in the pull request:
https://github.com/apache/flink/pull/5354#discussion_r164434005
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/resourcemanager/slotmanager/SlotManager.java
---
@@ -388,6 +378,8 @@ public boolean unregisterTaskManager(InstanceID
instanceId) {
public boolean reportSlotStatus(InstanceID instanceId, SlotReport
slotReport) {
checkInit();
+ LOG.info("Received slot report from instance " + instanceId +
'.');
--- End diff --
nit: I don't see a reason not to use slf4j placeholders here.
---
