You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ode.apache.org by "Rafal Rusin (JIRA)" <ji...@apache.org> on 2010/03/30 18:39:27 UTC
[jira] Created: (ODE-795) Speed up listAllProcesses
Speed up listAllProcesses
-------------------------
Key: ODE-795
URL: https://issues.apache.org/jira/browse/ODE-795
Project: ODE
Issue Type: Improvement
Components: BPEL Runtime
Affects Versions: 1.3.3
Reporter: Rafal Rusin
Assignee: Rafal Rusin
listAllProcesses in ProcessAndInstanceManagementImpl.java does a lot of queries to database in order to fetch instance summary. This causes transaction timeout. on ODE load.
Disabling instance summary is a workaround:
diff --git a/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/ProcessAndInstanceManagementImpl.java b/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/ProcessAndInstanceManagementImpl.java
index 47ada7a..0317864 100644
--- a/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/ProcessAndInstanceManagementImpl.java
+++ b/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/ProcessAndInstanceManagementImpl.java
@@ -805,6 +805,7 @@ public class ProcessAndInstanceManagementImpl implements InstanceManagement, Pro
depinfo.setDocument(pconf.getBpelDocument());
depinfo.setDeployDate(toCalendar(pconf.getDeployDate()));
depinfo.setDeployer(pconf.getDeployer());
+ /*
if (custom.includeInstanceSummary()) {
TInstanceSummary isum = info.addNewInstanceSummary();
genInstanceSummaryEntry(conn, isum.addNewInstances(), TInstanceStatus.ACTIVE, pconf);
@@ -814,7 +815,7 @@ public class ProcessAndInstanceManagementImpl implements InstanceManagement, Pro
genInstanceSummaryEntry(conn, isum.addNewInstances(), TInstanceStatus.SUSPENDED, pconf);
genInstanceSummaryEntry(conn, isum.addNewInstances(), TInstanceStatus.TERMINATED, pconf);
getInstanceSummaryActivityFailure(conn, isum, pconf);
- }
+ }*/
if (custom.includeDocumentLists()) {
TProcessInfo.Documents docinfo = info.addNewDocuments();
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Resolved: (ODE-795) Speed up listAllProcesses
Posted by "Rafal Rusin (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ODE-795?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Rafal Rusin resolved ODE-795.
-----------------------------
Resolution: Fixed
> Speed up listAllProcesses
> -------------------------
>
> Key: ODE-795
> URL: https://issues.apache.org/jira/browse/ODE-795
> Project: ODE
> Issue Type: Improvement
> Components: BPEL Runtime
> Affects Versions: 1.3.3
> Reporter: Rafal Rusin
> Assignee: Rafal Rusin
> Fix For: 1.3.5
>
>
> listAllProcesses in ProcessAndInstanceManagementImpl.java does a lot of queries to database in order to fetch instance summary. This causes transaction timeout. on ODE load.
> Disabling instance summary is a workaround:
> diff --git a/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/ProcessAndInstanceManagementImpl.java b/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/ProcessAndInstanceManagementImpl.java
> index 47ada7a..0317864 100644
> --- a/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/ProcessAndInstanceManagementImpl.java
> +++ b/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/ProcessAndInstanceManagementImpl.java
> @@ -805,6 +805,7 @@ public class ProcessAndInstanceManagementImpl implements InstanceManagement, Pro
> depinfo.setDocument(pconf.getBpelDocument());
> depinfo.setDeployDate(toCalendar(pconf.getDeployDate()));
> depinfo.setDeployer(pconf.getDeployer());
> + /*
> if (custom.includeInstanceSummary()) {
> TInstanceSummary isum = info.addNewInstanceSummary();
> genInstanceSummaryEntry(conn, isum.addNewInstances(), TInstanceStatus.ACTIVE, pconf);
> @@ -814,7 +815,7 @@ public class ProcessAndInstanceManagementImpl implements InstanceManagement, Pro
> genInstanceSummaryEntry(conn, isum.addNewInstances(), TInstanceStatus.SUSPENDED, pconf);
> genInstanceSummaryEntry(conn, isum.addNewInstances(), TInstanceStatus.TERMINATED, pconf);
> getInstanceSummaryActivityFailure(conn, isum, pconf);
> - }
> + }*/
>
> if (custom.includeDocumentLists()) {
> TProcessInfo.Documents docinfo = info.addNewDocuments();
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (ODE-795) Speed up listAllProcesses
Posted by "Hudson (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ODE-795?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12851500#action_12851500 ]
Hudson commented on ODE-795:
----------------------------
Integrated in ODE-1.x #87 (See [http://hudson.zones.apache.org/hudson/job/ODE-1.x/87/])
: Speedup listAllProcesses + test case
> Speed up listAllProcesses
> -------------------------
>
> Key: ODE-795
> URL: https://issues.apache.org/jira/browse/ODE-795
> Project: ODE
> Issue Type: Improvement
> Components: BPEL Runtime
> Affects Versions: 1.3.3
> Reporter: Rafal Rusin
> Assignee: Rafal Rusin
>
> listAllProcesses in ProcessAndInstanceManagementImpl.java does a lot of queries to database in order to fetch instance summary. This causes transaction timeout. on ODE load.
> Disabling instance summary is a workaround:
> diff --git a/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/ProcessAndInstanceManagementImpl.java b/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/ProcessAndInstanceManagementImpl.java
> index 47ada7a..0317864 100644
> --- a/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/ProcessAndInstanceManagementImpl.java
> +++ b/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/ProcessAndInstanceManagementImpl.java
> @@ -805,6 +805,7 @@ public class ProcessAndInstanceManagementImpl implements InstanceManagement, Pro
> depinfo.setDocument(pconf.getBpelDocument());
> depinfo.setDeployDate(toCalendar(pconf.getDeployDate()));
> depinfo.setDeployer(pconf.getDeployer());
> + /*
> if (custom.includeInstanceSummary()) {
> TInstanceSummary isum = info.addNewInstanceSummary();
> genInstanceSummaryEntry(conn, isum.addNewInstances(), TInstanceStatus.ACTIVE, pconf);
> @@ -814,7 +815,7 @@ public class ProcessAndInstanceManagementImpl implements InstanceManagement, Pro
> genInstanceSummaryEntry(conn, isum.addNewInstances(), TInstanceStatus.SUSPENDED, pconf);
> genInstanceSummaryEntry(conn, isum.addNewInstances(), TInstanceStatus.TERMINATED, pconf);
> getInstanceSummaryActivityFailure(conn, isum, pconf);
> - }
> + }*/
>
> if (custom.includeDocumentLists()) {
> TProcessInfo.Documents docinfo = info.addNewDocuments();
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (ODE-795) Speed up listAllProcesses
Posted by "Tammo van Lessen (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ODE-795?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Tammo van Lessen updated ODE-795:
---------------------------------
Fix Version/s: 1.3.5
> Speed up listAllProcesses
> -------------------------
>
> Key: ODE-795
> URL: https://issues.apache.org/jira/browse/ODE-795
> Project: ODE
> Issue Type: Improvement
> Components: BPEL Runtime
> Affects Versions: 1.3.3
> Reporter: Rafal Rusin
> Assignee: Rafal Rusin
> Fix For: 1.3.5
>
>
> listAllProcesses in ProcessAndInstanceManagementImpl.java does a lot of queries to database in order to fetch instance summary. This causes transaction timeout. on ODE load.
> Disabling instance summary is a workaround:
> diff --git a/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/ProcessAndInstanceManagementImpl.java b/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/ProcessAndInstanceManagementImpl.java
> index 47ada7a..0317864 100644
> --- a/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/ProcessAndInstanceManagementImpl.java
> +++ b/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/ProcessAndInstanceManagementImpl.java
> @@ -805,6 +805,7 @@ public class ProcessAndInstanceManagementImpl implements InstanceManagement, Pro
> depinfo.setDocument(pconf.getBpelDocument());
> depinfo.setDeployDate(toCalendar(pconf.getDeployDate()));
> depinfo.setDeployer(pconf.getDeployer());
> + /*
> if (custom.includeInstanceSummary()) {
> TInstanceSummary isum = info.addNewInstanceSummary();
> genInstanceSummaryEntry(conn, isum.addNewInstances(), TInstanceStatus.ACTIVE, pconf);
> @@ -814,7 +815,7 @@ public class ProcessAndInstanceManagementImpl implements InstanceManagement, Pro
> genInstanceSummaryEntry(conn, isum.addNewInstances(), TInstanceStatus.SUSPENDED, pconf);
> genInstanceSummaryEntry(conn, isum.addNewInstances(), TInstanceStatus.TERMINATED, pconf);
> getInstanceSummaryActivityFailure(conn, isum, pconf);
> - }
> + }*/
>
> if (custom.includeDocumentLists()) {
> TProcessInfo.Documents docinfo = info.addNewDocuments();
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira