You are viewing a plain text version of this content. The canonical link for it is here.

Posted to jira@kafka.apache.org by GitBox <gi...@apache.org> on 2020/09/23 07:08:39 UTC

[GitHub] [kafka] chia7712 commented on pull request #9262: MINOR: Fix log message when tasks directory is cleaned manually

chia7712 commented on pull request #9262:
URL: https://github.com/apache/kafka/pull/9262#issuecomment-697177819


   @cadonna 
   
   ```
   org.apache.kafka.streams.processor.internals.StateDirectoryTest.shouldLogStateDirCleanerMessage failed, log available in /home/chia7712/kafka/streams/build/reports/testOutput/org.apache.kafka.streams.processor.internals.StateDirectoryTest.shouldLogStateDirCleanerMessage.test.stdout
   
   org.apache.kafka.streams.processor.internals.StateDirectoryTest > shouldLogStateDirCleanerMessage FAILED
       java.lang.AssertionError:
       Expected: a collection containing a string ending with "ms has elapsed (cleanup delay is 0ms)."
            but: was empty
           at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
           at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:6)
           at org.apache.kafka.streams.processor.internals.StateDirectoryTest.shouldLogStateDirCleanerMessage(StateDirectoryTest.java:569)
   ```
   
   the new test introduced by this PR is failed. The failed test does not update the current time of ```MockTime``` so the state directory can't be cleanup. I have filed a ticket for that failed test (https://issues.apache.org/jira/browse/KAFKA-10514).


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