WildFly
  1. WildFly
  2. WFLY-1383

Disable pooling of stateless EJBs by default

    Details

    • Type: Enhancement Enhancement
    • Status: Resolved (View Workflow)
    • Priority: Major Major
    • Resolution: Done
    • Affects Version/s: 8.0.0.Alpha1
    • Fix Version/s: 8.0.0.Final
    • Component/s: EJB
    • Labels:
      None
    • Similar Issues:
      Show 10 results 

      Description

      A certain use case has shown that it's perhaps better to disable pooling of EJB stateless bean, by default. In AS7 we have always allowed disabling the pooling but we hadn't made it the default since we thought it might affect applications which have SLSBs with heavy @PostConstruct.

      A recent discussion has suggested that we disable the pooling by default and individual applications can enable it and set the relevant pool size themselves. The reasoning has been that, the default pool size that we ship with, anyway will have to be tweaked by the users since we can't really guess what an ideal pool size is.

        Gliffy Diagrams

          Activity

          Hide
          jaikiran pai added a comment -

          Pull request sent

          Show
          jaikiran pai added a comment - Pull request sent
          Hide
          Cheng Fang added a comment -

          Will manually take the changes to the 3 files from Jaikiran's PR:

          build/src/main/resources/configuration/subsystems/ejb3.xml
          testsuite/integration/basic/src/test/java/org/jboss/as/test/integration/ejb/management/deployments/ManagedStatelessBean.java
          testsuite/integration/basic/src/test/java/org/jboss/as/test/integration/ejb/management/deployments/NoTimerStatelessBean.java

          Changes to testsuite/integration/basic/src/test/java/org/jboss/as/test/integration/ejb/pool/lifecycle/PooledEJBLifecycleTestCase.java is obsoleted by fixes to WFLY-1506 (Fix PooledEJBLifecycleTestCase.tesMDB and the whole testcase in general).

          Show
          Cheng Fang added a comment - Will manually take the changes to the 3 files from Jaikiran's PR: build/src/main/resources/configuration/subsystems/ejb3.xml testsuite/integration/basic/src/test/java/org/jboss/as/test/integration/ejb/management/deployments/ManagedStatelessBean.java testsuite/integration/basic/src/test/java/org/jboss/as/test/integration/ejb/management/deployments/NoTimerStatelessBean.java Changes to testsuite/integration/basic/src/test/java/org/jboss/as/test/integration/ejb/pool/lifecycle/PooledEJBLifecycleTestCase.java is obsoleted by fixes to WFLY-1506 (Fix PooledEJBLifecycleTestCase.tesMDB and the whole testcase in general).
          Hide
          Cheng Fang added a comment -

          When testing with original master build, had 4 failures. Wait for a few days till all cts tests are passing and some pending issues are resolved.

          ./build.sh clean install -DallTests -fae

          wildfly-ts-integ-basic:
          /Users/cfang/dev/wildfly/testsuite/integration/basic/target/surefire-reports/org.jboss.as.test.integration.ejb.mdb.dynamic.DynamicMessageListenerTestCase.txt

          wildfly-ts-integ-xts:
          /Users/cfang/dev/wildfly/testsuite/integration/xts/target/surefire-reports/org.jboss.as.test.xts.wsba.participantcompletion.client.BAParticipantCompletionTestCase.txt
          Cannot deploy: compensatable-test.war

          wildfly-ts-integ-rbac:
          /Users/cfang/dev/wildfly/testsuite/integration/rbac/target/surefire-reports/org.jboss.as.test.integration.mgmt.access.PropertiesRoleMappingTestCase.txt
          Could not start container
          Managed server was not started within [60] s

          wildfly-ts-integ-manualmode:
          /Users/cfang/dev/wildfly/testsuite/integration/manualmode/target/surefire-reports/org.jboss.as.test.manualmode.ws.ReloadWSDLPublisherTestCase.txt
          Cannot deploy: jaxws-manual-pojo.war

          Show
          Cheng Fang added a comment - When testing with original master build, had 4 failures. Wait for a few days till all cts tests are passing and some pending issues are resolved. ./build.sh clean install -DallTests -fae wildfly-ts-integ-basic: /Users/cfang/dev/wildfly/testsuite/integration/basic/target/surefire-reports/org.jboss.as.test.integration.ejb.mdb.dynamic.DynamicMessageListenerTestCase.txt wildfly-ts-integ-xts: /Users/cfang/dev/wildfly/testsuite/integration/xts/target/surefire-reports/org.jboss.as.test.xts.wsba.participantcompletion.client.BAParticipantCompletionTestCase.txt Cannot deploy: compensatable-test.war wildfly-ts-integ-rbac: /Users/cfang/dev/wildfly/testsuite/integration/rbac/target/surefire-reports/org.jboss.as.test.integration.mgmt.access.PropertiesRoleMappingTestCase.txt Could not start container Managed server was not started within [60] s wildfly-ts-integ-manualmode: /Users/cfang/dev/wildfly/testsuite/integration/manualmode/target/surefire-reports/org.jboss.as.test.manualmode.ws.ReloadWSDLPublisherTestCase.txt Cannot deploy: jaxws-manual-pojo.war

            People

            • Assignee:
              Cheng Fang
              Reporter:
              jaikiran pai
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development