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

Posted to issues@maven.apache.org by GitBox <gi...@apache.org> on 2022/01/08 15:51:07 UTC

[GitHub] [maven] MartinKanters commented on pull request #639: [MNG-7310] Using the plexusContainer to prevent loading lifecycles from defined by extensions from other submodules.

MartinKanters commented on pull request #639:
URL: https://github.com/apache/maven/pull/639#issuecomment-1008035176


   > RU sure you don't need to perform lookup on each call (instead to keep the map in final member?) This is a singleton, so the map you lookup in constructor, once inited, would not change....
   
   Ah, then it is indeed best to lookup the map once and store it in an member variable. Good suggestion!


-- 
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: issues-unsubscribe@maven.apache.org

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