Re: svn commit: r487759 - /geronimo/server/trunk/modules/geronimo-kernel/src/main/java/org/apache/geronimo/gbean/runtime/GBeanOperation.java

[anita kulshreshtha <a_...@yahoo.com>]

Dims,
   I do not understand how/why this change fixes the problem discussed
here. The GBeanOperation was not (and need not be) a serializable
class. The error message is about GOperationInfo, which does have a
SUID. 
http://www.nabble.com/Re%3A-Trunk-runtime-error-GBeanInstanceState--deserializing-GBeanState-p7899716.html

   For an interim period GOpearationInfo SUID was wrong. Because of
which the published SNAPSHOTs were incompatible with the local class.
Am I missing something?

Thanks
Anita

 
--- dims@apache.org wrote:

> Author: dims
> Date: Fri Dec 15 21:14:55 2006
> New Revision: 487759
> 
> URL: http://svn.apache.org/viewvc?view=rev&rev=487759
> Log:
> Fix for the InvalidClassException i was having (and others were
> having sporadically). Problem is similar to the one in GERONIMO-1020.
> Unless the exact same jar is used for ser/de-ser, we get
> InvalidClassException, if you build the
> geronimo-kernel-xxx-SNAPSHOT.jar on your box and the maven2 pulls
> another SNAPSHOT jar from the remote repo, you are %&^*&ed
> 
> Modified:
>    
>
geronimo/server/trunk/modules/geronimo-kernel/src/main/java/org/apache/geronimo/gbean/runtime/GBeanOperation.java
> 
> Modified:
>
geronimo/server/trunk/modules/geronimo-kernel/src/main/java/org/apache/geronimo/gbean/runtime/GBeanOperation.java
> URL:
>
http://svn.apache.org/viewvc/geronimo/server/trunk/modules/geronimo-kernel/src/main/java/org/apache/geronimo/gbean/runtime/GBeanOperation.java?view=diff&rev=487759&r1=487758&r2=487759
>
==============================================================================
> ---
>
geronimo/server/trunk/modules/geronimo-kernel/src/main/java/org/apache/geronimo/gbean/runtime/GBeanOperation.java
> (original)
> +++
>
geronimo/server/trunk/modules/geronimo-kernel/src/main/java/org/apache/geronimo/gbean/runtime/GBeanOperation.java
> Fri Dec 15 21:14:55 2006
> @@ -22,6 +22,7 @@
>  import java.util.Collections;
>  import java.util.Iterator;
>  import java.util.List;
> +import java.io.Serializable;
>  
>  import org.apache.geronimo.gbean.DynamicGBean;
>  import org.apache.geronimo.gbean.DynamicGOperationInfo;
> @@ -32,7 +33,9 @@
>  /**
>   * @version $Rev$ $Date$
>   */
> -public final class GBeanOperation {
> +public final class GBeanOperation implements Serializable {
> +    private static final long serialVersionUID =
> -5185515581104192977L;
> +    
>      private final GBeanInstance gbeanInstance;
>      private final String name;
>      private final List parameterTypes;
> 
> 
> 



 
____________________________________________________________________________________
Yahoo! Music Unlimited
Access over 1 million songs.
http://music.yahoo.com/unlimited