Change in vdsm[master]: stomp: client side subscription

danken at redhat.com danken at redhat.com
Fri May 15 09:14:13 UTC 2015


Dan Kenigsberg has posted comments on this change.

Change subject: stomp: client side subscription
......................................................................


Patch Set 20:

(1 comment)

https://gerrit.ovirt.org/#/c/36368/20//COMMIT_MSG
Commit Message:

Line 9: In this patch we introduce concept of subscription for client
Line 10: perspective. We move queuing functionality out of AsyncDispatcher to
Line 11: frame_handler. New StompRpcClient class is responsible for sending
Line 12: subscriptions and ClientRpcTransportAdapter class adds 'reply-to' header
Line 13: using subscriptions id.
In this patch we introduce the concept of client subscription, which allows multiple clients.

Each client subscribes to its own destination. When processed, the responses to this client's requests are posted to that destination only.

We move queuing functionality out of AsyncDispatcher to
frame_handler. introduce new subscription classes (the existing one has never been used) ...

(try to rephrase and add info)
Line 14: 
Line 15: 
Line 16: Change-Id: I8e43658f1cebd637ea3abf4396d388afa041ae71
Line 17: Signed-off-by: Saggi Mizrahi <smizrahi at redhat.com>


-- 
To view, visit https://gerrit.ovirt.org/36368
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I8e43658f1cebd637ea3abf4396d388afa041ae71
Gerrit-PatchSet: 20
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Dima Kuznetsov <dkuznets at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Yaniv Bronheim <ybronhei at redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list