You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@kafka.apache.org by "mjsax (via GitHub)" <gi...@apache.org> on 2024/09/11 18:02:41 UTC

[PR] KAFKA-17524: Add timeout to StreamThreadTest.shouldReturnErrorIfProducerInstanceIdNotInitialized [kafka]

mjsax opened a new pull request, #17167:
URL: https://github.com/apache/kafka/pull/17167

   (no comment)


-- 
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: jira-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] KAFKA-17524: Add timeout to StreamThreadTest.shouldReturnErrorIfProducerInstanceIdNotInitialized [kafka]

Posted by "mjsax (via GitHub)" <gi...@apache.org>.
mjsax closed pull request #17167: KAFKA-17524: Add timeout to StreamThreadTest.shouldReturnErrorIfProducerInstanceIdNotInitialized
URL: https://github.com/apache/kafka/pull/17167


-- 
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: jira-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] KAFKA-17524: Add timeout to StreamThreadTest.shouldReturnErrorIfProducerInstanceIdNotInitialized [kafka]

Posted by "mjsax (via GitHub)" <gi...@apache.org>.
mjsax commented on PR #17167:
URL: https://github.com/apache/kafka/pull/17167#issuecomment-2345055691

   @mumrah -- SG. Let me add this to all tests in this category (there is a few more...)


-- 
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: jira-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] KAFKA-17524: Add timeout to StreamThreadTest.shouldReturnErrorIfProducerInstanceIdNotInitialized [kafka]

Posted by "mumrah (via GitHub)" <gi...@apache.org>.
mumrah commented on PR #17167:
URL: https://github.com/apache/kafka/pull/17167#issuecomment-2345015978

   Another here:
   
   https://github.com/apache/kafka/actions/runs/10818099338
   
   Test: StreamThreadTest > shouldNotCloseTaskAndRemoveFromTaskManagerIfProducerWasFencedWhileProcessing(boolean, boolean) > "shouldNotCloseTaskAndRemoveFromTaskManagerIfProducerWasFencedWhileProcessing(boolean, boolean).stateUpdaterEnabled=true, processingThreadsEnabled=true"
   Duration: 44m7s
   Raw line: 2024-09-11T21:05:34.3655550Z Gradle Test Run :streams:test > Gradle Test Executor 52 > StreamThreadTest > shouldNotCloseTaskAndRemoveFromTaskManagerIfProducerWasFencedWhileProcessing(boolean, boolean) > "shouldNotCloseTaskAndRemoveFromTaskManagerIfProducerWasFencedWhileProcessing(boolean, boolean).stateUpdaterEnabled=true, processingThreadsEnabled=true" STARTED
   
   


-- 
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: jira-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] KAFKA-17524: Add timeout to StreamThreadTest.shouldReturnErrorIfProducerInstanceIdNotInitialized [kafka]

Posted by "mumrah (via GitHub)" <gi...@apache.org>.
mumrah commented on PR #17167:
URL: https://github.com/apache/kafka/pull/17167#issuecomment-2344821912

   @mjsax I saw another seemingly stalled test in this class shouldNotCloseTaskAndRemoveFromTaskManagerIfPidMappingIsInvalidInCommitTransactionWhenCommitting https://issues.apache.org/jira/browse/KAFKA-17524?focusedCommentId=17881072&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-17881072
   
   should we add a timeout at the class level to be safe?


-- 
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: jira-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] KAFKA-17524: Add timeout to StreamThreadTest.shouldReturnErrorIfProducerInstanceIdNotInitialized [kafka]

Posted by "mumrah (via GitHub)" <gi...@apache.org>.
mumrah commented on PR #17167:
URL: https://github.com/apache/kafka/pull/17167#issuecomment-2346612112

   Something weird is definitely happening in this test class 🤔 
   
   Latest build of this PR has these two getting stuck:
   
   * shouldLogAndRecordSkippedMetricForDeserializationException 
   * shouldNotCloseTaskAndRemoveFromTaskManagerIfProducerGotFencedInCommitTransactionWhenCommitting
   
   


-- 
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: jira-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] KAFKA-17524: Add timeout to StreamThreadTest.shouldReturnErrorIfProducerInstanceIdNotInitialized [kafka]

Posted by "mjsax (via GitHub)" <gi...@apache.org>.
mjsax commented on PR #17167:
URL: https://github.com/apache/kafka/pull/17167#issuecomment-2346760103

   Thanks. Let's try this one instead: https://github.com/apache/kafka/pull/17180


-- 
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: jira-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org