[ 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: (was: 8.0.8) 8.0.9 > 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.9 > > > 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.3.4#803005)