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