[GitHub] [kafka] guozhangwang commented on pull request #8682: KAFKA-10011: Remove task id from lockedTaskDirectories during handleLostAll

[GitBox <gi...@apache.org>]

guozhangwang commented on pull request #8682:
URL: https://github.com/apache/kafka/pull/8682#issuecomment-630515587


   I agree with @ableegoldman here, after the `while (taskIdIterator.hasNext()` loop we can see if there are still remaining tasks, and then log an WARN similar to the end of `handleRevocation` before clearing them:
   
   ```
   if (!remainingPartitions.isEmpty()) {
               log.warn("The following partitions {} are missing from the task partitions. It could potentially " +
                            "due to race condition of consumer detecting the heartbeat failure, or the tasks " +
                            "have been cleaned up by the handleAssignment callback.", remainingPartitions);
           }
   ```


----------------------------------------------------------------
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:
users@infra.apache.org