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 <vvola...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Asaf Shakarchi <a...@redhat.com>
Gerrit-Reviewer: Ayal Baron <aba...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Liron Aravot <lara...@redhat.com>
Gerrit-Reviewer: Michael Kublin <mkub...@redhat.com>
Gerrit-Reviewer: Moti Asayag <masa...@redhat.com>
Gerrit-Reviewer: Vered Volansky <vvola...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to