Michael Kublin has posted comments on this change.
Change subject: core : Master reconstruct leads to inconsistent state (#800839)
......................................................................
Patch Set 1: (1 inline comment)
....................................................
File
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java
Line 552: String host =
TransactionSupport.executeInScope(TransactionScopeOption.Suppress, new
TransactionMethod<String>() {
Actually not, if we will fail an exception will be thrown at any case.
Behaviour of suppress is similar to RequiredNew for parent transaction, except
that no transaction will be opened as child.
--
To view, visit http://gerrit.ovirt.org/4862
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I93b1e8931fc68b1233dd562a3776ca29494d5976
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Michael Kublin <mkub...@redhat.com>
Gerrit-Reviewer: Ayal Baron <aba...@redhat.com>
Gerrit-Reviewer: Haim Ateya <hat...@redhat.com>
Gerrit-Reviewer: Michael Kublin <mkub...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
