Change in ovirt-engine[master]: engine: Disable global transaction at ExtendSANStorageDomain
mkublin at redhat.com
mkublin at redhat.com
Sun Feb 3 13:42:48 UTC 2013
Michael Kublin has submitted this change and it was merged.
Change subject: engine: Disable global transaction at ExtendSANStorageDomain
......................................................................
engine: Disable global transaction at ExtendSANStorageDomain
During investigation on some bug, I noticed that we have too many transaction time out
exceptions on ExtendSANStorageDomainCommand.
The problem is that following command is trying to open a global transaction, this is a mistake,
transaction is a global resource and should be kept as short as possible.
The change is:
1. Disable global transaction
2. Start to use compensation in order to keep flow correct
Change-Id: I78c4c43c259a58c746caa0402bf5beb004e9a8c2
Signed-off-by: Michael Kublin <mkublin at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ExtendSANStorageDomainCommand.java
1 file changed, 34 insertions(+), 18 deletions(-)
Approvals:
Michael Kublin: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/11648
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I78c4c43c259a58c746caa0402bf5beb004e9a8c2
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
More information about the Engine-commits
mailing list