You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@aries.apache.org by GitBox <gi...@apache.org> on 2022/10/01 15:19:27 UTC

[GitHub] [aries-rsa] ponziani commented on pull request #43: fix ConcurrentModificationException in ServiceExportsRepository

ponziani commented on PR #43:
URL: https://github.com/apache/aries-rsa/pull/43#issuecomment-1264396030

   Hi Tim,
   
   Thanks for your quick response. I'm absolutely no expert on the matter, but from reading [this SO answer](https://stackoverflow.com/a/16180833/4937772) I learnt that the current code (using exportsMap.get(sref) in a for-loop) is asking for a ConcurrentModificationException.
   
   > In access-ordered linked hash maps, merely querying the map with get is a structural modification.
   
   Maybe other improvements are possible as well, but for us this little improvement managed to get rid of the stacktrace.


-- 
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: dev-unsubscribe@aries.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org