ableegoldman commented on a change in pull request #9028:
URL: https://github.com/apache/kafka/pull/9028#discussion_r456101096



##########
File path: 
streams/src/test/java/org/apache/kafka/streams/integration/AbstractResetIntegrationTest.java
##########
@@ -265,7 +265,7 @@ public void 
shouldNotAllowToResetWhenIntermediateTopicAbsent() throws Exception
     public void testResetWhenLongSessionTimeoutConfiguredWithForceOption() 
throws Exception {

Review comment:
       Thanks for looking into this! It definitely doesn't sound good that some 
of these tests don't pass with SSL. Maybe there's just some additional setup 
that's needed for these tests? 🤔
   
   Anyways, we don't need to solve all that in this PR. We can revisit the 
issue once this is merged




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


Reply via email to