Yaniv Bronhaim has posted comments on this change.

Change subject: jsonrpc cli: allow client to return compatible results to xmlrpc
......................................................................


Patch Set 2: Code-Review-1

(1 comment)

https://gerrit.ovirt.org/#/c/47923/2//COMMIT_MSG
Commit Message:

Line 13: 
Line 14: The compat parameter allows us to return a value
Line 15: that is compatible with the way xmlrpc is parsed
Line 16: and we'll be able to remove this once we stop
Line 17: supporting xmlrpc API.
so please call the new parameter - xml_compatible=True\False and add a comment 
in code "# REQUIRED_FOR:" with explanation
Line 18: 
Line 19: Until then using this intermediate step is the most
Line 20: straight-forward way and saves us the unnecessary
Line 21: hassle of re-translating the bridge work


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id0bbbb46f622e60824cf5d25670fd16889f4fc51
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yeela Kaplan <[email protected]>
Gerrit-Reviewer: Jenkins CI
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

Reply via email to