[jboss-dev-forums] [Design of Clustering on JBoss (Clusters/JBoss)] - Re: Transaction Sticky LB policy for 4.2/trunk

[galder.zamarreno@jboss.com]

Back once again (like the renegade master :)) 

trunk and 4.2.x differ slightly when it comes to org.jboss.invocation.MarshalledInvocation class.

in 4.2.x, MarshalledInvocation(Invocation invocation) constructor does:

...
      this.transient_payload = invocation.transient_payload;
      this.invocationContext = invocation.invocationContext;
...

whereas in trunk, the transient payload is not copied over to the MarshalledInvocation.

The discrepancy comes from:

http://viewvc.jboss.org/cgi-bin/viewvc.cgi/jbossas/branches/Branch_4_2/server/src/main/org/jboss/invocation/MarshalledInvocation.java?r1=38132&r2=41001

as part of:

"JBAS-2436 - Commiting Pluggable Serialization into JBAS 4.0" (http://jira.jboss.com/jira/browse/JBAS-2436)

Whether the transient payload is copied over to the MarshalledInvocation is not very important because eventually, writeExternal won't write it. However, why would you copy over a transient value over to an invocation that will be marshalled and which will not include transient values??

This discrepancy is making the tests that run in trunk not work in 4.2.x. I can workaround this but I don't think this type of discrepancies are good.

Thoughts? Worth a JIRA?

View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4112393#4112393

Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4112393