JBossDeveloper

Ok, I have made some changes and I think we have solved it, these are 3 fold and can be found on my branch https://github.com/andytaylor/hornetq/tree/serialization_issues.

firsly i have changed back DiscoveryGroupConfiguration so the class description is the same as 2.2, I have then added write/readObject methods, if jgroups is used then we serialize the BroadcastEndpointFactoryConfiguration manually, if its UDPBroadcastGroupConfiguration then we dont do anything on the write and rebuild it on the read, this means that this will work with both 2.2 and 2.3 with no changes or flags needed.

secondly, i had to move Pair.java back to its old location, i gues moving the API packages is a no no if they are needed for serialization, we should do this for 3.0.

lastly, fixed an issue with PacketDecoder on server locator, it needs to be transient and re created in the readObject method.

Howard is testing this now.

It looks good...

just one minor observation:

if you could just fix the alignments on these new statements you are adding:

if(localBindPort < 0)

^^ it would be better to keep the space as everything else. We don't have checkstyle now but we are that on our wish list.. better to avoiding adding future checkstyle errors

Also: I will ask you a favour: Please add a big javadoc warning on ServerLocator that any new control field needs to be transient as that class is serialized over the wire