youngkermit8-coder opened a new pull request, #1755: URL: https://github.com/apache/rocketmq-dashboard/pull/1755
## Summary - add credential-scoped AdminClient cache eviction - use the old credential reference when an Apache instance changes credentials - preserve the existing endpoint-wide eviction used for endpoint removal or replacement ## Root cause and impact The AdminClient cache key includes both the normalized NameServer endpoint and an authentication identity. However, the instance credential-change path called the endpoint-wide `release` overload, which shut down every cached identity at that endpoint. When instances share an endpoint with different credentials, updating one instance could therefore interrupt or force reconnection for unrelated instances. The new overload removes only the old `(endpoint, credential reference)` entry, while endpoint deletion retains the existing all-identities behavior. Fixes #1754. ## Validation - Reproduced on current `rocketmq-studio`: the unrelated credential identity was evicted and client creation count was 3 instead of 2 - `mvn -B -ntp -Dtest=MqAdminExtFactoryTest,RuntimeAdminClientResolverTest,InstanceServiceTest,ApacheAclReadServiceTest test` — 70 tests passed - focused credential-scoped eviction regression — passed - Maven Checkstyle — 0 violations - `mvn -B -ntp -DskipTests package` — passed - `git diff --check` — passed -- 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]
