Change in vdsm[master]: BZ#754445 - Seperate granting from callback emitting

abaron at redhat.com abaron at redhat.com
Sat Dec 3 22:02:32 UTC 2011


Ayal Baron has posted comments on this change.

Change subject: BZ#754445 - Seperate granting from callback emitting
......................................................................


Patch Set 6: I would prefer that you didn't submit this

(4 inline comments)

....................................................
Commit Message
Line 13: On the other hand granting the request has to be synchronious and
also fix synchronious

Line 14: transactional to prevent races with cancel. To keep both demands I split
also, you're repeating yourself (Granting has to happen... On the other hand granting ...)

....................................................
File vdsm/storage/resourceManager.py
Line 452:         # Code in a nested function's body may access but not rebind
Actually I don't see the difference between list and queue in this respect so I'm not sure the comment should change.
however the apostrophe in wouldn't is after the 'n', not before.

Line 470:         return resource.get()
if for some reason resource queue is empty then the thread would hang here forever. the previous version of the code handled this state

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic12d170e64399e37a555960c03804778ad7d053b
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Haim Ateya <hateya at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>


More information about the vdsm-patches mailing list