[ 
https://issues.apache.org/jira/browse/OWB-496?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12998293#comment-12998293
 ] 

Mark Struberg commented on OWB-496:
-----------------------------------

patch looks really good - I like the idea of just setting an own special 
ClassLoaderProvider which a) uses a ThreadLocal to work around the concurrency 
problems and b) provides our own plugable logic.
please go on and commit the patch. We will fine tune it later together if 
needed.

> Don't replace the ProxyFactory classloaderProvider without the intention to 
> do so
> ---------------------------------------------------------------------------------
>
>                 Key: OWB-496
>                 URL: https://issues.apache.org/jira/browse/OWB-496
>             Project: OpenWebBeans
>          Issue Type: Bug
>          Components: Context and Scopes
>    Affects Versions: 1.1.0
>            Reporter: David Jencks
>            Assignee: Gurkan Erdogdu
>             Fix For: 1.1.0
>
>         Attachments: OWB-496.patch
>
>
> Currently JavassistProxyFactory.getProxyClass() replaces the 
> ProxyFactory.classloaderProvider on any exception with a classloaderProvider 
> that is very unlikely to work better than the default.  Setting the 
> classLoaderProvider should be a matter of intentional configuration, not 
> flailing around after an unexpected exception.

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to