[jboss-jira] [Red Hat JIRA] (WFLY-13871) Re-implement correct suspend/resume behaviour for EJB client

[ https://issues.redhat.com/browse/WFLY-13871?page=com.atlassian.jira.plugi... ] Richard Achmatowicz commented on WFLY-13871: -------------------------------------------- Some further comments on this issue. After looking in other parts of the codebase, I see that there *already is* a mechanism in place for advising EJB clients of suspend/resume events on the server, but it seems like it is just not working correctly. I'm going to try to describe the mechanism, which is related to https://issues.redhat.com/browse/WFLY-7207 Here is a brief summary: when an invocation for EJB component C arrives at the server, it passes through EjbSuspendInterceptor. This interceptor checks the ControlPoint for component C to see if the invocation is allowed to proceed. If it is not allowed to proceed, before rejecting the invocation, a service EjbSuspendHandlerService is checked to see if *it* thinks the invocation should be allowed to proceed. EjbSuspendHandlerService decides whether or not an invocation should proceed based on two counts that it keeps: a count of active invocations, and a count of active transactions. If a transaction is active, it lets the invocation proceed, as transaction completion may depend on the invocation completing. EjbSuspendHandlerService will only report that an invocation may not proceed when (roughly speaking) there are no longer any active transactions on the server. This behaviour is controlled by an attribute, gracefulTransactionShutdown, of the EJB subsyetem. If gracefulTransactionShutdown is true, the EjbSuspendInterceptor will wait until all active transactions have completed; if false, it will not. EjbSuspendInterceptor  takes information from the SuspendController (i.e. it is a listener for suspend/resume events) and takes information from LocalTransactionContext (i.e. it listens for transaction create and transaction completion events). It also keeps track of active invocations, via calls from EjbSuspendInterceptor. In addition to this, EjbSuspendHandlerService, once it receives a suspend event from the SuspendController, it will tell the LocalTransactionContext to prohibit the creation of any new transactions which are created with a call to beginTransaction which has the parameter failOnSuspend == true. It will also wait for any active transactions to complete (either commit or rollback) and when the last one completes, it will do two things: notify the SuspendController that, as a ServerActivity kistener, it is ready to suspend, and secondly, it suspends the DeploymentRepositoryService by calling DeploymentRepository.suspend().   The DeploymentRepository, when suspended, sends out deploymentSuspended() notifications to all DeploymentRepository listeners for all currently deployed modules, and sends out deploymentAvailable() followed by deploymentSuspended() notifications for any subsequently deployed modules. When the DeploymentRepository is later resumed (by the EjbSuspendHandlerService), it sends out deploymentResumed () notifications to all DeploymentRepository listeners. These deploymentSuspended() and deploymentResumed() notifications get converted into moduleUnavailable() and moduleUnavailable() messages sent out to all EJB clients connected to the server. So, this arrangement should be providing the correct sequence of moduleUnAvailable and moduleAvailable messages to EJB clients, but something is going wrong somewhere. So, to summarise, the current PR will be thrown out and i'll focus on fixing the existing mechanism.           -- This message was sent by Atlassian Jira (v7.13.8#713008)