[OWB-357] WebbeansFinder should index first on ClassLoader, not singleton type

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: M4
• Fix Version/s: 1.0.0-alpha-1
• Environment: tiered classloader
• Assignee: Gurkan Erdogdu
• Reporter: Eric Covener
• Created: 16/Apr/10 00:27
• Updated: 07/May/10 21:02
• Resolved: 07/May/10 21:02

Description

I'm finding WebbeansFinder a little difficult to work with since it is unable to easily clear all instances a / the current classloader.

I would like to change the map to have the primary index on the classloader from:

private static Map<String, Map<ClassLoader, Object>> singletonMap = new HashMap<String, Map<ClassLoader, Object>>();
to
private static Map<ClassLoader, Map<String, Object>> singletonMap = new HashMap<ClassLoader, Map<String, Object>>();

Currently, this only really breaks removeInstance(String name) which AFAICT has no callers. This would be replaced by e.g.

clearInstance(ClassLoader cl)

And most current callers of clearInstances() would I think prefer to just clear an individual classloader.

Opinions welcome, and curious if anyone wants to preserve a removeInstance(String name) method.