Oved Ourfali has submitted this change and it was merged.

Change subject: xmlrpc: marshalling of Long type
......................................................................


xmlrpc: marshalling of Long type

The change let people use 64bit numerical type (long) with xmlrpc.
Jsonrpc already supports long type.

There is xmlrpc extension i8 which let people use long type but we do
not want to enable it due to backward compatibility reasons.

In order to use this code developer needs to add method name to
SUPPORTED_METHODS_FOR_LONG_CONVERSION collection. This reduces overhead
of type fixing only to known methods. Long type is normalized to string
to let older vdsms understand the value.


Change-Id: I97ffc2da3703dbf7ce58607b7461a02aea35f2bb
Signed-off-by: pkliczewski <[email protected]>
---
M 
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/xmlrpc/XmlRpcUtils.java
1 file changed, 35 insertions(+), 0 deletions(-)

Approvals:
  Piotr Kliczewski: Verified
  Martin Sivák: Looks good to me, but someone else must approve
  Yair Zaslavsky: Looks good to me, but someone else must approve
  Oved Ourfali: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I97ffc2da3703dbf7ce58607b7461a02aea35f2bb
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Piotr Kliczewski <[email protected]>
Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]>
Gerrit-Reviewer: Martin Sivák <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: Piotr Kliczewski <[email protected]>
Gerrit-Reviewer: Roy Golan <[email protected]>
Gerrit-Reviewer: Saggi Mizrahi <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to