leedongkyu0407 opened a new pull request, #1953: URL: https://github.com/apache/stormcrawler/pull/1953
Closes #1951 ## Summary Replace the O(n) linear scan in `getConfiguredProxy()` with an O(1) map lookup. ## Changes - Added `proxyLookupMap` field (HashMap keyed on normalized proxy identity) - Built the map in both `configure()` overloads after proxies are loaded - Added `proxyKey()` helper that normalizes `protocol` and `address` to lowercase, consistent with `ProxyUtils.isSameProxy()` - Replaced the for-loop in `getConfiguredProxy()` with a map lookup ## Notes - No behavior change - Key includes credentials to match `ProxyUtils.isSameProxy()` semantics - `MultiProxyManagerTest`: 13/13 passed - `ProtocolTest.testBlocking` failure is pre-existing and unrelated to this change -- 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]
