svn commit: r824522 - /ode/branches/APACHE_ODE_1.X/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/BpelRuntimeContextImpl.java

[boisvert]

Author: boisvert
Date: Mon Oct 12 22:23:04 2009
New Revision: 824522

URL: http://svn.apache.org/viewvc?rev=824522&view=rev
Log:
Don't cancel invoke check directly; let it be scheduled and discarded quietly 
instead.  Better for performance, especially on non-MVCC databases

Modified:
    
ode/branches/APACHE_ODE_1.X/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/BpelRuntimeContextImpl.java

Modified: 
ode/branches/APACHE_ODE_1.X/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/BpelRuntimeContextImpl.java
URL: 
http://svn.apache.org/viewvc/ode/branches/APACHE_ODE_1.X/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/BpelRuntimeContextImpl.java?rev=824522&r1=824521&r2=824522&view=diff
==============================================================================
--- 
ode/branches/APACHE_ODE_1.X/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/BpelRuntimeContextImpl.java
 (original)
+++ 
ode/branches/APACHE_ODE_1.X/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/BpelRuntimeContextImpl.java
 Mon Oct 12 22:23:04 2009
@@ -1275,10 +1275,14 @@
         MessageExchangeDAO dao = 
_dao.getConnection().getMessageExchange(mexId);
         dao.release(_bpelProcess.isCleanupCategoryEnabled(instanceSucceeded, 
CLEANUP_CATEGORY.MESSAGES) );
 
-        // Canceling invocation check job
+        // We used to cancel the invoke check job here but it turns out
+        // it creates more contention on the ODE_JOB table.  It's better
+        // just to let the job get scheduled and discarded quietly
+        /*
         String jobId = dao.getProperty("invokeCheckJobId");
         if (jobId != null)
             _bpelProcess._engine._contexts.scheduler.cancelJob(jobId);
+        */
     }