[jboss-dev-forums] [Design of EJB 3.0] - Re: EJBTHREE-1454 Encapsulate Container infomation in TO/VO

[jaikiran]

"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#4179227

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