jackrabbit-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Christophe Lombart (JIRA)" <j...@apache.org>
Subject [jira] Resolved: (JCR-1869) Make lazy loading proxy callback Serializable
Date Wed, 26 Nov 2008 20:22:44 GMT

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

Christophe Lombart resolved JCR-1869.
-------------------------------------

       Resolution: Fixed
    Fix Version/s: 1.6.0

Patch applied. Thanks


> Make lazy loading proxy callback Serializable
> ---------------------------------------------
>
>                 Key: JCR-1869
>                 URL: https://issues.apache.org/jira/browse/JCR-1869
>             Project: Jackrabbit
>          Issue Type: Improvement
>          Components: jackrabbit-ocm
>            Reporter: Stephane Landelle
>            Assignee: Christophe Lombart
>             Fix For: 1.6.0
>
>         Attachments: AbstractLazyLoader.java, BeanLazyLoader.java, CollectionLazyLoader.java
>
>
> Hello (probably Christophe :) ),
> It would be nice to have the CGLib callbacks Serializable, because if they're not, the
proxies are not either, even if the target is.
> I've seen that you've made BeanLazyLoader Serializable. Wouldn't be better to have AbstractLazyLoader
Serializable, so CollectionLazyLoader is too?
> What I've done is :
> *) declaring AbstractLazyLoader as Serializable
> *) declaring non-Serializable resources such as Session as volatile
> *) throwing an IllegalStateException in BeanLazyLoader.fetch and CollectionLazyLoader.fetch
if the Session is null. This case can only happen if the proxy has been serialized without
the volatile Session, and it doesn't make sense to lazy load the target then.
> BTW, I realized I didn't clean up the beanClassDescriptor in BeanLazyLoader.cleanUp,
so I corrected this.
> I'll attach the modified classes.
> Sincerely,
> St├ęphane Landelle

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


Mime
View raw message