[jira] Created: (PLUTO-555) Weaken classloading assumptions for castor and resource bundles

Weaken classloading assumptions for castor and resource bundles
---------------------------------------------------------------

                 Key: PLUTO-555
                 URL: https://issues.apache.org/jira/browse/PLUTO-555
             Project: Pluto
          Issue Type: Bug
          Components: portal driver
    Affects Versions: 1.1.6
            Reporter: David Jencks
            Assignee: David Jencks


While working to get pluto working in geronimo with a one-classloader-per-jar osgi-like classloading framework I discovered a couple of assumptions.

1. Castor is in the same classloader as pluto-descriptor-impl.  Instead we can give castor the pluto-descriptor-impl classloader explicitly
2. the tooltips resource bundle is in the same classloader as pluto-portal-driver.  Instead we can use the thread context classloader.  This should be OK since this code is executed in a jaavaee context, so the web app classloader is the thread context classloader.  Picking the correct locale is still a problem.

Patch is for 1.1.6, need to see how it relates to trunk (and 1.2, and 2)

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

     [ https://issues.apache.org/jira/browse/PLUTO-555?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

David Jencks updated PLUTO-555:
-------------------------------

    Attachment: PLUTO-555.diff

patch for 1.1.6

> Weaken classloading assumptions for castor and resource bundles
> ---------------------------------------------------------------
>
>                 Key: PLUTO-555
>                 URL: https://issues.apache.org/jira/browse/PLUTO-555
>             Project: Pluto
>          Issue Type: Bug
>          Components: portal driver
>    Affects Versions: 1.1.6
>            Reporter: David Jencks
>            Assignee: David Jencks
>         Attachments: PLUTO-555.diff
>
>
> While working to get pluto working in geronimo with a one-classloader-per-jar osgi-like classloading framework I discovered a couple of assumptions.
> 1. Castor is in the same classloader as pluto-descriptor-impl.  Instead we can give castor the pluto-descriptor-impl classloader explicitly
> 2. the tooltips resource bundle is in the same classloader as pluto-portal-driver.  Instead we can use the thread context classloader.  This should be OK since this code is executed in a jaavaee context, so the web app classloader is the thread context classloader.  Picking the correct locale is still a problem.
> Patch is for 1.1.6, need to see how it relates to trunk (and 1.2, and 2)

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

    [ https://issues.apache.org/jira/browse/PLUTO-555?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12701709#action_12701709 ]

David Jencks commented on PLUTO-555:
------------------------------------

Applied to branches/1.1.x in rev 767697.

> Weaken classloading assumptions for castor and resource bundles
> ---------------------------------------------------------------
>
>                 Key: PLUTO-555
>                 URL: https://issues.apache.org/jira/browse/PLUTO-555
>             Project: Pluto
>          Issue Type: Bug
>          Components: portal driver
>    Affects Versions: 1.1.6
>            Reporter: David Jencks
>            Assignee: David Jencks
>         Attachments: PLUTO-555.diff
>
>
> While working to get pluto working in geronimo with a one-classloader-per-jar osgi-like classloading framework I discovered a couple of assumptions.
> 1. Castor is in the same classloader as pluto-descriptor-impl.  Instead we can give castor the pluto-descriptor-impl classloader explicitly
> 2. the tooltips resource bundle is in the same classloader as pluto-portal-driver.  Instead we can use the thread context classloader.  This should be OK since this code is executed in a jaavaee context, so the web app classloader is the thread context classloader.  Picking the correct locale is still a problem.
> Patch is for 1.1.6, need to see how it relates to trunk (and 1.2, and 2)

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

    [ https://issues.apache.org/jira/browse/PLUTO-555?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12701727#action_12701727 ]

David Jencks commented on PLUTO-555:
------------------------------------

Only the ResourceBundle ToolTips problem appears possible in trunk .... castor appears to be gone.

> Weaken classloading assumptions for castor and resource bundles
> ---------------------------------------------------------------
>
>                 Key: PLUTO-555
>                 URL: https://issues.apache.org/jira/browse/PLUTO-555
>             Project: Pluto
>          Issue Type: Bug
>          Components: portal driver
>    Affects Versions: 1.1.6
>            Reporter: David Jencks
>            Assignee: David Jencks
>         Attachments: PLUTO-555.diff
>
>
> While working to get pluto working in geronimo with a one-classloader-per-jar osgi-like classloading framework I discovered a couple of assumptions.
> 1. Castor is in the same classloader as pluto-descriptor-impl.  Instead we can give castor the pluto-descriptor-impl classloader explicitly
> 2. the tooltips resource bundle is in the same classloader as pluto-portal-driver.  Instead we can use the thread context classloader.  This should be OK since this code is executed in a jaavaee context, so the web app classloader is the thread context classloader.  Picking the correct locale is still a problem.
> Patch is for 1.1.6, need to see how it relates to trunk (and 1.2, and 2)

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