Change in ovirt-engine[ovirt-engine-3.5]: xmlrpc: marshalling of Long type

oourfali at redhat.com oourfali at redhat.com
Tue Nov 18 12:11:31 UTC 2014


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 <piotr.kliczewski at gmail.com>
Bug-Url: https://bugzilla.redhat.com/1145621
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/xmlrpc/XmlRpcUtils.java
1 file changed, 36 insertions(+), 0 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I97ffc2da3703dbf7ce58607b7461a02aea35f2bb
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list