Change in vdsm[master]: jsonrpc: Run JsonRpcServer requests in their own thread

danken at redhat.com danken at redhat.com
Sun Jan 27 16:04:04 UTC 2013


Dan Kenigsberg has submitted this change and it was merged.

Change subject: jsonrpc: Run JsonRpcServer requests in their own thread
......................................................................


jsonrpc: Run JsonRpcServer requests in their own thread

Currently it uses the transport thread.
This change creates a clear separation of control flow.

This makes sure there is a single sync point for all json-rpc requests
and that handling a request never blocks the transport for processing
messages.

Change-Id: Ifccf3fd0e3b1bec7db475b2674cff09bb920738e
Signed-off-by: Saggi Mizrahi <smizrahi at redhat.com>
---
M tests/jsonRpcTests.py
M vdsm_api/BindingJsonRpc.py
M vdsm_api/jsonrpc/__init__.py
3 files changed, 106 insertions(+), 28 deletions(-)

Approvals:
  Dan Kenigsberg: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ifccf3fd0e3b1bec7db475b2674cff09bb920738e
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Adam Litke <agl at us.ibm.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>


More information about the vdsm-patches mailing list