You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@wink.apache.org by "Mike Rheinheimer (JIRA)" <ji...@apache.org> on 2009/11/04 23:29:32 UTC

[jira] Created: (WINK-227) need fallback for Class.forName in J2EE environments

need fallback for Class.forName in J2EE environments
----------------------------------------------------

                 Key: WINK-227
                 URL: https://issues.apache.org/jira/browse/WINK-227
             Project: Wink
          Issue Type: Bug
          Components: Common
    Affects Versions: 1.1
            Reporter: Mike Rheinheimer
             Fix For: 1.1


Overlooked a possible need in wink-common.  Looks like we need the same fix in org.apache.wink.common.internal.application.ApplicationFileLoader as was done in WINK-226.  Wink common does not yet have a dependency on Apache commons lang, which is what I used in WINK-226 since wink-server already had that dependency.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (WINK-227) need fallback for Class.forName in J2EE environments

Posted by "Hudson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/WINK-227?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12775088#action_12775088 ] 

Hudson commented on WINK-227:
-----------------------------

Integrated in Wink-Trunk-JDK1.5 #240 (See [http://hudson.zones.apache.org/hudson/job/Wink-Trunk-JDK1.5/240/])
    : use thread context classloader over system classloader in J2EE environments


> need fallback for Class.forName in J2EE environments
> ----------------------------------------------------
>
>                 Key: WINK-227
>                 URL: https://issues.apache.org/jira/browse/WINK-227
>             Project: Wink
>          Issue Type: Bug
>          Components: Common
>    Affects Versions: 1.1
>            Reporter: Mike Rheinheimer
>             Fix For: 1.1
>
>         Attachments: WINK-227.patch
>
>
> Overlooked a possible need in wink-common.  Looks like we need the same fix in org.apache.wink.common.internal.application.ApplicationFileLoader as was done in WINK-226.  Wink common does not yet have a dependency on Apache commons lang, which is what I used in WINK-226 since wink-server already had that dependency.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (WINK-227) need fallback for Class.forName in J2EE environments

Posted by "Hudson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/WINK-227?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12775386#action_12775386 ] 

Hudson commented on WINK-227:
-----------------------------

Integrated in Wink-Trunk-JDK1.5-itests #15 (See [http://hudson.zones.apache.org/hudson/job/Wink-Trunk-JDK1.5-itests/15/])
    : use thread context classloader over system classloader in J2EE environments


> need fallback for Class.forName in J2EE environments
> ----------------------------------------------------
>
>                 Key: WINK-227
>                 URL: https://issues.apache.org/jira/browse/WINK-227
>             Project: Wink
>          Issue Type: Bug
>          Components: Common
>    Affects Versions: 1.1
>            Reporter: Mike Rheinheimer
>             Fix For: 1.1
>
>         Attachments: WINK-227.patch
>
>
> Overlooked a possible need in wink-common.  Looks like we need the same fix in org.apache.wink.common.internal.application.ApplicationFileLoader as was done in WINK-226.  Wink common does not yet have a dependency on Apache commons lang, which is what I used in WINK-226 since wink-server already had that dependency.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Closed: (WINK-227) need fallback for Class.forName in J2EE environments

Posted by "Mike Rheinheimer (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WINK-227?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mike Rheinheimer closed WINK-227.
---------------------------------


> need fallback for Class.forName in J2EE environments
> ----------------------------------------------------
>
>                 Key: WINK-227
>                 URL: https://issues.apache.org/jira/browse/WINK-227
>             Project: Wink
>          Issue Type: Bug
>          Components: Common
>    Affects Versions: 1.1
>            Reporter: Mike Rheinheimer
>             Fix For: 1.1
>
>         Attachments: WINK-227.patch
>
>
> Overlooked a possible need in wink-common.  Looks like we need the same fix in org.apache.wink.common.internal.application.ApplicationFileLoader as was done in WINK-226.  Wink common does not yet have a dependency on Apache commons lang, which is what I used in WINK-226 since wink-server already had that dependency.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (WINK-227) need fallback for Class.forName in J2EE environments

Posted by "Hudson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/WINK-227?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12775426#action_12775426 ] 

Hudson commented on WINK-227:
-----------------------------

Integrated in Wink-Trunk-JDK1.6-itests #16 (See [http://hudson.zones.apache.org/hudson/job/Wink-Trunk-JDK1.6-itests/16/])
    : use thread context classloader over system classloader in J2EE environments


> need fallback for Class.forName in J2EE environments
> ----------------------------------------------------
>
>                 Key: WINK-227
>                 URL: https://issues.apache.org/jira/browse/WINK-227
>             Project: Wink
>          Issue Type: Bug
>          Components: Common
>    Affects Versions: 1.1
>            Reporter: Mike Rheinheimer
>             Fix For: 1.1
>
>         Attachments: WINK-227.patch
>
>
> Overlooked a possible need in wink-common.  Looks like we need the same fix in org.apache.wink.common.internal.application.ApplicationFileLoader as was done in WINK-226.  Wink common does not yet have a dependency on Apache commons lang, which is what I used in WINK-226 since wink-server already had that dependency.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (WINK-227) need fallback for Class.forName in J2EE environments

Posted by "Mike Rheinheimer (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WINK-227?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mike Rheinheimer updated WINK-227:
----------------------------------

    Attachment: WINK-227.patch

> need fallback for Class.forName in J2EE environments
> ----------------------------------------------------
>
>                 Key: WINK-227
>                 URL: https://issues.apache.org/jira/browse/WINK-227
>             Project: Wink
>          Issue Type: Bug
>          Components: Common
>    Affects Versions: 1.1
>            Reporter: Mike Rheinheimer
>             Fix For: 1.1
>
>         Attachments: WINK-227.patch
>
>
> Overlooked a possible need in wink-common.  Looks like we need the same fix in org.apache.wink.common.internal.application.ApplicationFileLoader as was done in WINK-226.  Wink common does not yet have a dependency on Apache commons lang, which is what I used in WINK-226 since wink-server already had that dependency.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (WINK-227) need fallback for Class.forName in J2EE environments

Posted by "Mike Rheinheimer (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WINK-227?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mike Rheinheimer resolved WINK-227.
-----------------------------------

    Resolution: Fixed

Fixed in SVN rev 834183

> need fallback for Class.forName in J2EE environments
> ----------------------------------------------------
>
>                 Key: WINK-227
>                 URL: https://issues.apache.org/jira/browse/WINK-227
>             Project: Wink
>          Issue Type: Bug
>          Components: Common
>    Affects Versions: 1.1
>            Reporter: Mike Rheinheimer
>             Fix For: 1.1
>
>         Attachments: WINK-227.patch
>
>
> Overlooked a possible need in wink-common.  Looks like we need the same fix in org.apache.wink.common.internal.application.ApplicationFileLoader as was done in WINK-226.  Wink common does not yet have a dependency on Apache commons lang, which is what I used in WINK-226 since wink-server already had that dependency.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (WINK-227) need fallback for Class.forName in J2EE environments

Posted by "Mike Rheinheimer (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WINK-227?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mike Rheinheimer updated WINK-227:
----------------------------------

    Attachment:     (was: WINK-227.patch)

> need fallback for Class.forName in J2EE environments
> ----------------------------------------------------
>
>                 Key: WINK-227
>                 URL: https://issues.apache.org/jira/browse/WINK-227
>             Project: Wink
>          Issue Type: Bug
>          Components: Common
>    Affects Versions: 1.1
>            Reporter: Mike Rheinheimer
>             Fix For: 1.1
>
>         Attachments: WINK-227.patch
>
>
> Overlooked a possible need in wink-common.  Looks like we need the same fix in org.apache.wink.common.internal.application.ApplicationFileLoader as was done in WINK-226.  Wink common does not yet have a dependency on Apache commons lang, which is what I used in WINK-226 since wink-server already had that dependency.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (WINK-227) need fallback for Class.forName in J2EE environments

Posted by "Mike Rheinheimer (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WINK-227?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mike Rheinheimer updated WINK-227:
----------------------------------

    Attachment: WINK-227.patch

Changed from depending on apache commons lang ClassUtils to our own implementation in wink-common.  This way we can consolidate and control classloader priorities when getting system resources or loading classes.

Please see patch.  Did not add test due to complexity of classloader management and Ant's delegating classloader messing things up in a test like this anyway.  We'll have to rely on J2EE test environments, which is already part of the itests anyway.

> need fallback for Class.forName in J2EE environments
> ----------------------------------------------------
>
>                 Key: WINK-227
>                 URL: https://issues.apache.org/jira/browse/WINK-227
>             Project: Wink
>          Issue Type: Bug
>          Components: Common
>    Affects Versions: 1.1
>            Reporter: Mike Rheinheimer
>             Fix For: 1.1
>
>         Attachments: WINK-227.patch
>
>
> Overlooked a possible need in wink-common.  Looks like we need the same fix in org.apache.wink.common.internal.application.ApplicationFileLoader as was done in WINK-226.  Wink common does not yet have a dependency on Apache commons lang, which is what I used in WINK-226 since wink-server already had that dependency.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.