[GitHub] [pulsar] lifepuzzlefun commented on pull request #20196: [improve][broker] Improve knowBrokers update

2023-04-27 Thread via GitHub
lifepuzzlefun commented on PR #20196: URL: https://github.com/apache/pulsar/pull/20196#issuecomment-1526967491 @AnonHxy hi i found some previous fix on the ConcurrentModificationException when on `knownBrokers` field. and in my opinion this method will have a chance to execute

[GitHub] [pulsar] lifepuzzlefun commented on pull request #20196: [improve][broker] Improve knowBrokers update

2023-04-27 Thread via GitHub
lifepuzzlefun commented on PR #20196: URL: https://github.com/apache/pulsar/pull/20196#issuecomment-1526930780 the code here. it seems a fix to avoid a problem. context here https://github.com/apache/pulsar/pull/16690 -- This is an automated message from the Apache Git Service. To