Liron Aravot has posted comments on this change.

Change subject: core : Preview snapshot - NPE after engine crash
......................................................................


Patch Set 2:

why add try-catch which is such an expensive command when you can avoid it? 
this patch will be reverted anyway as AsyncTaskManager new handling would be 
added and that's the reason for the separate patches (the fix to the 
compensation shouldn't be reverted anyway otherwise we will encounter this 
exception somewhere else).

--
To view, visit http://gerrit.ovirt.org/9073
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia4ddecc990fbca23fb98e8ab42cb1eb039e1fb59
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Ayal Baron <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Liron Aravot <[email protected]>
Gerrit-Reviewer: Maor Lipchuk <[email protected]>
Gerrit-Reviewer: Michael Kublin <[email protected]>
Gerrit-Reviewer: Vered Volansky <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to