Mike Kolesnik has posted comments on this change.

Change subject: engine: Remove duplicate properties of VmTemplate
......................................................................


Patch Set 4:

(1 comment)

....................................................
File 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmTemplate.java
Line 222
Line 223
Line 224
Line 225
Line 226
There is no @JsonIgnore on this field in the super class, this means that 
whenever the template object will get serialized by json it will also include 
the disks list.

This would seem to re-introduce https://bugzilla.redhat.com/808866

Is this intentional?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I765da987d21ddf7153715322be9b9156b2a0ddca
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Betak <[email protected]>
Gerrit-Reviewer: Martin Betak <[email protected]>
Gerrit-Reviewer: Mike Kolesnik <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: Roy Golan <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to