Hey all,

My first MyFaces patch:

 https://issues.apache.org/jira/browse/MYFACES-3581

Small issue on deserialization.  JDK proxies need similar workaround code in 
the deserialization part.  Likely went unneeded, but when the ManagedBeans hold 
an EJB ref from OpenEJB then you run into it.  Not sure why this never surfaced 
in Geronimo, but patch adds in the code we use on the OpenEJB side to do 
deserialization that is JDK proxy-aware.

Does sort of break the intended architecture.  Added some comments on that in 
the JIRA.

We could go a couple different ways on that.  Probably the only way that 
matches the current approach is to add the method to the classloading 
abstraction.  Downside is it would break all existing implementations.

Maybe we could commit as-is with it wired up to 
Thread.currentThread().getClassLoader() and abstract it out later.


-David

Reply via email to