[
https://issues.apache.org/jira/browse/TOMEE-1618?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Richard Zowalla updated TOMEE-1618:
-----------------------------------
Fix Version/s: 8.0.12
(was: 8.0.11)
> Replace three register maps in Container in favour of one
> ---------------------------------------------------------
>
> Key: TOMEE-1618
> URL: https://issues.apache.org/jira/browse/TOMEE-1618
> Project: TomEE
> Issue Type: Improvement
> Components: TomEE Core Server
> Affects Versions: 7.0.0
> Reporter: Alex
> Assignee: Alex
> Priority: Trivial
> Labels: newbie
> Fix For: 8.0.12
>
>
> The class org.apache.tomee.embedded.Container currently uses three internal
> ly managed maps to register following information:
> moduleIds, appContexts and appInfos.
> The key used in each of the map is the name of the application.
> The suggested improvement is to introduce a multimap. I think however, it
> would be cleaner to define an inner class that can hold the information
> above, and have one map that maps the name to an instance of this class.
> Drawback is an extra null check, but overall I think this better.
> Additionally, this class does not have many unittests so I'd like to add
> those as well.
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
