[weld-issues] [JBoss JIRA] Commented: (WELD-867) bean implementations should be serializable

2011-03-14 Thread Mark Struberg (JIRA)

[ 
https://issues.jboss.org/browse/WELD-867?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12587903#comment-12587903
 ] 

Mark Struberg commented on WELD-867:


Hi!

To clarify this issue a bit, the problem is not that Beans are not serializable 
(this could be worked around by using the passivationId) but that they will get 
stored in CreationalContexts which is not serializable anymore afterwards. So 
either make the Beans serializable, or also use the passivationId in your 
CreationalContext impl.

> bean implementations should be serializable
> ---
>
> Key: WELD-867
> URL: https://issues.jboss.org/browse/WELD-867
> Project: Weld
>  Issue Type: Feature Request
>  Components: Built-in beans
>Affects Versions: 1.1.0.Final
>Reporter: Gerhard Petracek
>Priority: Blocker
> Fix For: 1.2.0.CR1
>
>
> for custom scope implementations it's required that beans are serializable.
> -> one of the following classes should implement java.io.Serializable
> https://github.com/weld/core/blob/master/impl/src/main/java/org/jboss/weld/bean/RIBean.java
> https://github.com/weld/core/blob/master/impl/src/main/java/org/jboss/weld/bean/AbstractBean.java
> https://github.com/weld/core/blob/master/impl/src/main/java/org/jboss/weld/bean/ManagedBean.java

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
___
weld-issues mailing list
weld-issues@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/weld-issues


[weld-issues] [JBoss JIRA] Commented: (WELD-867) bean implementations should be serializable

2011-03-20 Thread Stuart Douglas (JIRA)

[ 
https://issues.jboss.org/browse/WELD-867?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12589205#comment-12589205
 ] 

Stuart Douglas commented on WELD-867:
-

I don't think the beans should be marked as Serializable.

The correct approach is to use the passivationId, actually Serializing the 
beans will only cause problems/bugs (not to mention being very inefficient). 

> bean implementations should be serializable
> ---
>
> Key: WELD-867
> URL: https://issues.jboss.org/browse/WELD-867
> Project: Weld
>  Issue Type: Feature Request
>  Components: Built-in beans, Clustering, Scopes & Contexts
>Affects Versions: 1.1.0.Final
>Reporter: Gerhard Petracek
> Fix For: 1.2.0.Beta1
>
>
> for custom scope implementations it's required that beans are serializable.
> -> one of the following classes should implement java.io.Serializable
> https://github.com/weld/core/blob/master/impl/src/main/java/org/jboss/weld/bean/RIBean.java
> https://github.com/weld/core/blob/master/impl/src/main/java/org/jboss/weld/bean/AbstractBean.java
> https://github.com/weld/core/blob/master/impl/src/main/java/org/jboss/weld/bean/ManagedBean.java

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
___
weld-issues mailing list
weld-issues@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/weld-issues


[weld-issues] [JBoss JIRA] Commented: (WELD-867) bean implementations should be serializable

2011-03-20 Thread Mark Struberg (JIRA)

[ 
https://issues.jboss.org/browse/WELD-867?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12589212#comment-12589212
 ] 

Mark Struberg commented on WELD-867:


Stu, the idea was to use the passivationId internally in the Beans and not to 
serialize the all the fields in those beans ;)

> bean implementations should be serializable
> ---
>
> Key: WELD-867
> URL: https://issues.jboss.org/browse/WELD-867
> Project: Weld
>  Issue Type: Feature Request
>  Components: Built-in beans, Clustering, Scopes & Contexts
>Affects Versions: 1.1.0.Final
>Reporter: Gerhard Petracek
> Fix For: 1.2.0.Beta1
>
>
> for custom scope implementations it's required that beans are serializable.
> -> one of the following classes should implement java.io.Serializable
> https://github.com/weld/core/blob/master/impl/src/main/java/org/jboss/weld/bean/RIBean.java
> https://github.com/weld/core/blob/master/impl/src/main/java/org/jboss/weld/bean/AbstractBean.java
> https://github.com/weld/core/blob/master/impl/src/main/java/org/jboss/weld/bean/ManagedBean.java

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
___
weld-issues mailing list
weld-issues@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/weld-issues


[weld-issues] [JBoss JIRA] Commented: (WELD-867) bean implementations should be serializable

2011-03-20 Thread Stuart Douglas (JIRA)

[ 
https://issues.jboss.org/browse/WELD-867?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12589214#comment-12589214
 ] 

Stuart Douglas commented on WELD-867:
-

Doh, of course. 

Even so I am still not sure sure it is a good idea, as extensions that rely on 
this behaviour will not be truly portable. 

> bean implementations should be serializable
> ---
>
> Key: WELD-867
> URL: https://issues.jboss.org/browse/WELD-867
> Project: Weld
>  Issue Type: Feature Request
>  Components: Built-in beans, Clustering, Scopes & Contexts
>Affects Versions: 1.1.0.Final
>Reporter: Gerhard Petracek
> Fix For: 1.2.0.Beta1
>
>
> for custom scope implementations it's required that beans are serializable.
> -> one of the following classes should implement java.io.Serializable
> https://github.com/weld/core/blob/master/impl/src/main/java/org/jboss/weld/bean/RIBean.java
> https://github.com/weld/core/blob/master/impl/src/main/java/org/jboss/weld/bean/AbstractBean.java
> https://github.com/weld/core/blob/master/impl/src/main/java/org/jboss/weld/bean/ManagedBean.java

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
___
weld-issues mailing list
weld-issues@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/weld-issues


[weld-issues] [JBoss JIRA] Commented: (WELD-867) bean implementations should be serializable

2011-03-21 Thread Pete Muir (JIRA)

[ 
https://issues.jboss.org/browse/WELD-867?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12589294#comment-12589294
 ] 

Pete Muir commented on WELD-867:


Personally I don't think there is any reason to make Bean implement 
serializable, it's better to use the spec defined mechanism of 
getPassivationCapableBean

> bean implementations should be serializable
> ---
>
> Key: WELD-867
> URL: https://issues.jboss.org/browse/WELD-867
> Project: Weld
>  Issue Type: Feature Request
>  Components: Built-in beans, Clustering, Scopes & Contexts
>Affects Versions: 1.1.0.Final
>Reporter: Gerhard Petracek
> Fix For: 1.2.0.Beta1
>
>
> for custom scope implementations it's required that beans are serializable.
> -> one of the following classes should implement java.io.Serializable
> https://github.com/weld/core/blob/master/impl/src/main/java/org/jboss/weld/bean/RIBean.java
> https://github.com/weld/core/blob/master/impl/src/main/java/org/jboss/weld/bean/AbstractBean.java
> https://github.com/weld/core/blob/master/impl/src/main/java/org/jboss/weld/bean/ManagedBean.java

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
___
weld-issues mailing list
weld-issues@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/weld-issues


[weld-issues] [JBoss JIRA] Commented: (WELD-867) bean implementations should be serializable

2011-03-21 Thread Mark Struberg (JIRA)

[ 
https://issues.jboss.org/browse/WELD-867?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12589295#comment-12589295
 ] 

Mark Struberg commented on WELD-867:


As I said in my initial comment: the main problem is not the serialization of 
Bean itself, but serialization of CreationalContexts containing those Beans! 
CreationalContext itself is not Serializable yet and there is no passivationId 
for them of course...

> bean implementations should be serializable
> ---
>
> Key: WELD-867
> URL: https://issues.jboss.org/browse/WELD-867
> Project: Weld
>  Issue Type: Feature Request
>  Components: Built-in beans, Clustering, Scopes & Contexts
>Affects Versions: 1.1.0.Final
>Reporter: Gerhard Petracek
> Fix For: 1.2.0.Beta1
>
>
> for custom scope implementations it's required that beans are serializable.
> -> one of the following classes should implement java.io.Serializable
> https://github.com/weld/core/blob/master/impl/src/main/java/org/jboss/weld/bean/RIBean.java
> https://github.com/weld/core/blob/master/impl/src/main/java/org/jboss/weld/bean/AbstractBean.java
> https://github.com/weld/core/blob/master/impl/src/main/java/org/jboss/weld/bean/ManagedBean.java

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
___
weld-issues mailing list
weld-issues@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/weld-issues