From Yaniv Bronhaim ybronhei@redhat.com:
Yaniv Bronhaim has posted comments on this change.
Change subject: rpc: propagate executor issue to the engine ......................................................................
Patch Set 23: Code-Review-1
(1 comment)
https://gerrit.ovirt.org/#/c/74005/23/lib/vdsm/rpc/http.py File lib/vdsm/rpc/http.py:
Line 281: self.requestHandler = RequestHandlerClass Line 282: Line 283: # TODO provide proper limit for this queue Line 284: # pylint: disable=no-value-for-parameter Line 285: self.queue = TaskQueue(sys.maxint) are you sure about this code? this seems like sys.maxint will be used as the queue name here. odd.. Line 286: Line 287: def add(self, connected_socket, socket_address): Line 288: self.queue.put((connected_socket, socket_address)) Line 289:
vdsm-patches@lists.fedorahosted.org