Created attachment 26095 [details] Patch that provides lookup functionality and resolves explained issue 1* Support for @Resource("lookup") @EJB(lookup) @WebServiceRef("lookup") and via XML .... <lookup-name> 2* Currently DefaultInstanceManager looks for <injection-target> element's <injection-target-class> name for getting injections from web.xml. But it must also look at class hierarchy. This patch also resolves this issue.
Created attachment 26096 [details] Test for the lookup
There is a commit on the 7.0 branch that says it's related to this bug entry: http://svn.apache.org/viewvc?view=revision&revision=1826828 This commit changes the default factory from dbcp to dbcp2, but since dbcp2 support isn't present until version 8, I don't think this is a correct change. A quick glance at the other changes in the commit looks like they're just whitespace changes.
(In reply to Shawn Heisey from comment #2) > There is a commit on the 7.0 branch that says it's related to this bug entry: > > http://svn.apache.org/viewvc?view=revision&revision=1826828 > > This commit changes the default factory from dbcp to dbcp2, but since dbcp2 > support isn't present until version 8, I don't think this is a correct > change. > > A quick glance at the other changes in the commit looks like they're just > whitespace changes. I filed bug 62316 for this issue.
Apologies for the long delay in applying this. Thanks for the patch. Fixed in: - trunk for 9.0.9 onwards - 8.5.x for 8.5.32 onwards - 8.0.x for 8.0.53 onwards - 7.0.x for 7.0.89 onwards