JiayaoS commented on PR #22785:
URL: https://github.com/apache/kafka/pull/22785#issuecomment-4911295637

   With replication.factor=3 and min.isr=2, the test takes:
   ```xml
   <testcase name="testAuthorizationWithTopicExisting()" 
classname="kafka.api.AuthorizerIntegrationTest" time="57.812"/
   ```
   
   With replication.factor=1 and min.isr=1, the test takes:
   ```xml
   <testcase name="testAuthorizationWithTopicExisting()" 
classname="kafka.api.AuthorizerIntegrationTest" time="25.215"/>
   ```
   
   Other test cases did not show a significant reduction in runtime, so they 
are not included.
   
   Since the assert 
https://github.com/apache/kafka/blob/1db1ae701c0cecbf5d868fae2f376246410511c9/core/src/test/scala/integration/kafka/api/AuthorizerIntegrationTest.scala#L4247
 only checks for authorization-related errors, the previous share topic create 
fail and retries did not cause the test to fail.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to