Vered Volansky has posted comments on this change.

Change subject: core: Changed illegal error msg to locked(#840304)
......................................................................


Patch Set 6: Verified

There is no bug in the current patch that I know of. If you think otherwise 
please let me know, if I were aware of it I wouldn't have committed it to begin 
with. As noted in the previous commit message, this patch fixed an open bug as 
well as a related regression. As a result of your previous comments I decided 
to separate the two since they are too loosely coupled. This one id the solely 
the bug, the next one will handle the regression in the files I've pulled out 
of the patch. Please comment on the current patch as it is.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2edb7b8c946589f3bbad251d533704fe856bc175
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Vered Volansky <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Asaf Shakarchi <[email protected]>
Gerrit-Reviewer: Ayal Baron <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Liron Aravot <[email protected]>
Gerrit-Reviewer: Michael Kublin <[email protected]>
Gerrit-Reviewer: Moti Asayag <[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