Dan Kenigsberg has submitted this change and it was merged. Change subject: stomp: preparation of subscribtion class for subscription management ......................................................................
stomp: preparation of subscribtion class for subscription management So far subscription class was not used but it was designed to handle unsubscribe in __del__ method which is not the safest idea. We have decided to prepare it for proper unsubscription logic implement in: https://gerrit.ovirt.org/#/c/39969 Change-Id: I40cb609432c3d5a7d297e25647a152abdc8e86c5 Signed-off-by: pkliczewski <[email protected]> Reviewed-on: https://gerrit.ovirt.org/41068 Reviewed-by: Dan Kenigsberg <[email protected]> Continuous-Integration: Jenkins CI --- M lib/yajsonrpc/stomp.py 1 file changed, 25 insertions(+), 15 deletions(-) Approvals: Piotr Kliczewski: Verified Jenkins CI: Passed CI tests Dan Kenigsberg: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/41068 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I40cb609432c3d5a7d297e25647a152abdc8e86c5 Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Dima Kuznetsov <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Vinzenz Feenstra <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: Yeela Kaplan <[email protected]> Gerrit-Reviewer: [email protected] _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
