Change in ovirt-engine[master]: core: serialization/deserialization of SnapshotType

laravot at redhat.com laravot at redhat.com
Wed Feb 3 15:06:16 UTC 2016


Liron Aravot has submitted this change and it was merged.

Change subject: core: serialization/deserialization of SnapshotType
......................................................................


core: serialization/deserialization of SnapshotType

Because of the toString() implementation of SnapshotType (which is used
in returned messages) its deserialization is broken.
This patch fixes the serialization issue, as this code seems unneeded
to me I've removed it in change I8fb722dd15f0ccc2ac058765bfd6802a2a21134c.

This patch just fixes the serialization is it should be backported to the
3.6 branch and I'd like to minimize the changes and testing scope there.

Change-Id: Iee7192bd4e3bdf0b56ee57d4b25c5fcc17190be7
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1303840
Signed-off-by: Liron Aravot <laravot at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/Snapshot.java
1 file changed, 18 insertions(+), 0 deletions(-)

Approvals:
  Allon Mureinik: Looks good to me, but someone else must approve
  Moti Asayag: Looks good to me, approved
  Liron Aravot: Verified; Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iee7192bd4e3bdf0b56ee57d4b25c5fcc17190be7
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list