"ALRubinger" wrote : | | "jaikiran" wrote : | | I just noticed that there is a StatefulSessionInvokableContext (in proxy) which currently implements the InvokableContext. I guess, this should now implement the new InvokableSessionContext. Do you recommend that i change this too? | | | | Yep. | Okay, i included this in my changes. "ALRubinger" wrote : | If you can verify the clusteredsession tests in TestSuite are OK, than this is good for commit. | | Andrew, Sorry that this one took really long than what i expected. Mainly because of my lack of understanding on how to get the clustered testcases working in the testsuite. All that i learnt, while trying to get the clusteredsession testcases running, i have noted in this wiki http://wiki.jboss.org/wiki/DevEJB3RunClusteredTestCases. Might help someone else, or even me, the next time i try this :-) I started off with running the testsuite without including my changes (for EJBTHREE-1454) to get the tests passing and results comparable with the "EJB3 Plugin Against AS5 trunk" http://hudson.jboss.org/hudson/job/EJB3_Plugin_AS_TRUNK/. After a lot of (trial and error) iterations i was able to get them passing and results almost the same as that of hudson (i had a couple of more testcases failing without even including my changes). I then patched my local JBossAS (Trunk) with the EJBTHREE-1454 changes (followed the "Patch AS" section in this wiki http://wiki.jboss.org/wiki/en/DevEJB3NewPlugin) and ran the clusteredsession testcases again. No new failures/errors were introduced. So the patch looks OK. Andrew, Now that the testsuite too looks OK with these changes, do you want me to commit this? Futhermore, the poms that i have changed are: proxy module: Index: pom.xml | =================================================================== | --- pom.xml (revision 78909) | +++ pom.xml (working copy) | @@ -13,7 +13,7 @@ | <!-- Artifact Configuration --> | <groupId>org.jboss.ejb3</groupId> | <artifactId>jboss-ejb3-proxy</artifactId> | - <version>0.1.6-SNAPSHOT</version> | + <version>0.1.7-SNAPSHOT</version> | <name>JBoss EJB 3.0 Proxy</name> | <description>JBoss EJB3 Proxy Component</description> | <url>http://www.jboss.org/jbossejb3</url> | proxy-clustered module (to depend on the proxy's newer version): | Index: pom.xml | =================================================================== | --- pom.xml (revision 78909) | +++ pom.xml (working copy) | @@ -90,7 +90,7 @@ | <dependency> | <groupId>org.jboss.ejb3</groupId> | <artifactId>jboss-ejb3-proxy</artifactId> | - <version>0.1.2</version> | + <version>0.1.7-SNAPSHOT</version> | </dependency> | | <dependency> | core module pom (to depend on the newer versions of proxy and proxy-clustered): Index: pom.xml | =================================================================== | --- pom.xml (revision 78909) | +++ pom.xml (working copy) | @@ -365,13 +365,13 @@ | <dependency> | <groupId>org.jboss.ejb3</groupId> | <artifactId>jboss-ejb3-proxy</artifactId> | - <version>0.1.6-SNAPSHOT</version> | + <version>0.1.7-SNAPSHOT</version> | </dependency> | | <dependency> | <groupId>org.jboss.ejb3</groupId> | <artifactId>jboss-ejb3-proxy-clustered</artifactId> | - <version>0.1.3</version> | + <version>0.1.4-SNAPSHOT</version> | </dependency> | | <dependency> | plugin module pom (to integrate these newer version of proxy and proxy-clustered in the JBoss AS): Index: pom.xml | =================================================================== | --- pom.xml (revision 78909) | +++ pom.xml (working copy) | @@ -95,14 +95,14 @@ | <groupId>org.jboss.ejb3</groupId> | <artifactId>jboss-ejb3-proxy</artifactId> | <classifier>client</classifier> | - <version>0.1.6-SNAPSHOT</version> | + <version>0.1.7-SNAPSHOT</version> | </dependency> | | <dependency> | <groupId>org.jboss.ejb3</groupId> | <artifactId>jboss-ejb3-proxy-clustered</artifactId> | <classifier>client</classifier> | - <version>0.1.3</version> | + <version>0.1.4-SNAPSHOT</version> | </dependency> | | <dependency> | View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4179227#... Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&a...