chia7712 opened a new pull request #10050:
URL: https://github.com/apache/kafka/pull/10050


   ```ClusterConnectionStatesTest#testMultipleIPsWithUseAll``` assume the 
number of addresses is 3. That assumption is unstable since it can be changed 
in the future. For example, the addresses of ```kafka.apache.org``` is changed 
to 2 recently.
   
   It seems to me the test case is used to make sure all addresses (from 
resolving) are used when building connections. Hence, we can rewrite it by 
for-loop to make sure all addresses are used across connections.
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   


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