[Engine-devel] Redunant class of ValueObjectMap ?

[Itamar Heim]

> From: engine-devel-bounces at ovirt.org
[mailto:engine-devel-bounces at ovirt.org] On Behalf Of Roy Golan
> Sent: Thursday, December 22, 2011 17:53 PM
> To: Yair Zaslavsky
> Cc: engine-devel at ovirt.org
> Subject: Re: [Engine-devel] Redunant class of ValueObjectMap ?
> 
> On Thu 22 Dec 2011 05:50:29 PM IST, Yair Zaslavsky wrote:
> > Hi all,
> > Do we need org.ovirt.engine.core.common.queries.ValueObjectMap?
> > Can we use a Java map class directly instead (we had to use this with
SOAP WS in the past, prior to
> current rest-API days)?
> >
> > Yair

> I remember too it was for soap because the lack of support in maps.

valueObject* was only used for the SOAP serialization of the Object class.
The backend (engine core) is not supposed to use it anywhere (as in you
should be able to just delete all places using these classes).
Not supposed to need to change anything, since they are supposed to have
extra getters/setters using them just for the serialization sake.

(please add me as reviewer to the patch removing them).

Thanks,
   Itamar