Nir Soffer has posted comments on this change. Change subject: xmlrpc: do not block forever when waiting on incoming messages ......................................................................
Patch Set 5: (1 comment) https://gerrit.ovirt.org/#/c/42915/5//COMMIT_MSG Commit Message: Line 13: protocol and pass socket for processing. Line 14: Line 15: This patch changes how request queue is managed. We want to be sure that Line 16: when we stop only sentinel is queued so the xmlrpc server thread is Line 17: properly closed. The code is nice, but it does not fix blocking issue. The only change is dropping pending requests when stopping the server, and allowing a limit on the number of pending requests, which is not used yet. Please update the commit message. Line 18: Line 19: Line 20: Change-Id: I295e3099ea06d786741164e1f240f4662631bf8a -- To view, visit https://gerrit.ovirt.org/42915 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I295e3099ea06d786741164e1f240f4662631bf8a Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
