jiteshkhatri11 commented on PR #2138:
URL: https://github.com/apache/maven-resolver/pull/2138#issuecomment-6657983949

   I've updated the implementation based on the review feedback.
   
   - Replaced the mutable `ArrayList` collections with eagerly initialized 
`CopyOnWriteArrayList` instances.
   - Removed synchronization from the collection access methods.
   - Made the getter results unmodifiable to prevent external modification of 
internal state.
   - Added concurrent addition and iteration tests for both exceptions and 
dependency cycles.
   
   **Testing:** `mvn -pl maven-resolver-api test` — 96 tests passed, 0 
failures, 0 errors.
   
   Could you please review the updated changes? Thank you!


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