geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "David Jencks (JIRA)" <>
Subject [jira] Commented: (GERONIMO-2227) ENC Lookup Fix (include parent modules)
Date Thu, 27 Jul 2006 15:29:14 GMT
    [ ] 
David Jencks commented on GERONIMO-2227:

Maybe I'm missing something, but you patch appears to assign null to an already null variable.
 It also doesn't address the issue of looking in parents.  I also don't understand where the

-                    //moduleURI = linkName.substring(0, pos);

you are proposing to remove came from and think that it would be correct.  The idea is that
is you specify 


we will only look in that particular j2ee module, not some other that might have a similarly
named admin object.

Do you have an sample to show the problem you are trying to solve?

> ENC Lookup Fix (include parent modules)
> ---------------------------------------
>                 Key: GERONIMO-2227
>                 URL:
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>    Affects Versions: 1.1
>            Reporter: Aaron Mulder
>         Assigned To: David Jencks
>             Fix For: 1.1.1
>         Attachments: 2227-enc-include-parents.patch
> I ran across this while working on plugins that use resource-ref-like features to identify
a database connection pool, JMS destination, etc.
> The problem was that some naming searches were restricted to the current module only
when I felt all parents should be searched too.
> I would like David J to review this before I commit it, as my understanding of the code
in question is not terribly thorough.

This message is automatically generated by JIRA.
If you think it was sent incorrectly contact one of the administrators:
For more information on JIRA, see:


View raw message