Eli Mesika has submitted this change and it was merged.

Change subject: core: Cleanup VmStatic
......................................................................


core: Cleanup VmStatic

Addressed issues discovered by Eclipse IDE.

Note that the quotaId field that was defined twice (in VmBase and
VmStatic) was cleaned up, and now only appears in VmBase, together with
its annotation.

Change-Id: Ie7242569beb8b102318a4214a16fd7511b805442
Signed-off-by: Allon Mureinik <[email protected]>
---
M 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmBase.java
M 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmStatic.java
2 files changed, 3 insertions(+), 6 deletions(-)

Approvals:
  Eli Mesika: Verified; Looks good to me, approved
  Allon Mureinik: Verified; Looks good to me, but someone else must approve


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie7242569beb8b102318a4214a16fd7511b805442
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Maor Lipchuk <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to