Yes, if it doesn't affect a properly registered AdvancedExternalizer then we should only do this for 5.2.0.
On 3 May 2012, at 15:40, Vladimir Blagojevic wrote: > Is this change going to blow up serialization of our objects? People > might use different Infinispan micro version inadvertently.... > On 12-05-03 10:18 AM, Paul Ferraro wrote: >> Thanks for the clarification Galder. I'm in the process of creating >> Externalizers to optimize the serialization done by Infinispan and this >> clears up a lot of my confusion. >> >> It's ok by me if you make this change in 5.1.x. >> >> On 05/03/2012 02:23 AM, Galder Zamarreño wrote: >>> Re: https://issues.jboss.org/browse/ISPN-2029 >>> >>> Just a heads up as a result of an interesting discussion with Paul Ferraro >>> last night: >>> >>> So far Externalizer interface has extended Serializable and last night I >>> was trying to wonder why that was. >>> >>> As you guys might remember (https://docs.jboss.org/author/x/PwY5), there's >>> two type of externalizers: user friendly and advanced. >>> >>> The user friendly ones do not require any pre-registration or anything and >>> thanks to the ability of JBMAR to ship Externalizers to remote nodes, we >>> can support such use case. However, this requires Externalizer impls to be >>> Serializable or Externalizable somehow. >>> >>> However, that requirement is not there for advanced externalizers cos these >>> must be registered somehow, either via programmatic or XML configuration, >>> so in these case there's no shipping of Externalizer impls at all. >>> >>> So, I'm gonna remove 'implements Serializable' from >>> org.infinispan.marshall.Externalizer and instead require that any >>> user-friendly externalizers are marker as Serializable or similar. >>> >>> I'll be making this change in master, but what about 5.1.x? This change is >>> not urgent but clarifies what the externalizer serialization requirements >>> are. >>> >>> Cheers, >>> -- >>> Galder Zamarreño >>> Sr. Software Engineer >>> Infinispan, JBoss Cache >>> >> _______________________________________________ >> infinispan-dev mailing list >> [email protected] >> https://lists.jboss.org/mailman/listinfo/infinispan-dev > > _______________________________________________ > infinispan-dev mailing list > [email protected] > https://lists.jboss.org/mailman/listinfo/infinispan-dev -- Manik Surtani [email protected] twitter.com/maniksurtani Lead, Infinispan http://www.infinispan.org _______________________________________________ infinispan-dev mailing list [email protected] https://lists.jboss.org/mailman/listinfo/infinispan-dev
