Eli Mesika has posted comments on this change. Change subject: engine: Disable global trnasaction at ExtendSANStorageDomain ......................................................................
Patch Set 1: Looks good to me, but someone else must approve (3 inline comments) .................................................... Commit Message Line 5: CommitDate: 2013-02-03 12:56:01 +0200 Line 6: Line 7: engine: Disable global trnasaction at ExtendSANStorageDomain Line 8: Line 9: During investigation on some bug, I took attentins that we have to many transaction time out took attentins => noticed Line 10: exception on ExtendSANStorageDomainCommand. Line 11: The problem is that following command is trying to open a global transaction, these is a mistake, Line 12: transaction is a global resource and should be kept as short as possible. Line 13: The change is: Line 6: Line 7: engine: Disable global trnasaction at ExtendSANStorageDomain Line 8: Line 9: During investigation on some bug, I took attentins that we have to many transaction time out Line 10: exception on ExtendSANStorageDomainCommand. exception => exception s Line 11: The problem is that following command is trying to open a global transaction, these is a mistake, Line 12: transaction is a global resource and should be kept as short as possible. Line 13: The change is: Line 14: 1. Disable global transaction Line 7: engine: Disable global trnasaction at ExtendSANStorageDomain Line 8: Line 9: During investigation on some bug, I took attentins that we have to many transaction time out Line 10: exception on ExtendSANStorageDomainCommand. Line 11: The problem is that following command is trying to open a global transaction, these is a mistake, these=>this Line 12: transaction is a global resource and should be kept as short as possible. Line 13: The change is: Line 14: 1. Disable global transaction Line 15: 2. Start to use compensation in order to keep flow correct -- To view, visit http://gerrit.ovirt.org/11648 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I78c4c43c259a58c746caa0402bf5beb004e9a8c2 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Eli Mesika <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
