Change in ovirt-engine[master]: dal: Remove duplicate fields in VM row mappers
ofrenkel at redhat.com
ofrenkel at redhat.com
Mon Jul 7 11:05:26 UTC 2014
Omer Frenkel has submitted this change and it was merged.
Change subject: dal: Remove duplicate fields in VM row mappers
......................................................................
dal: Remove duplicate fields in VM row mappers
Removed duplicate fields that were specified both in AbstractVmRowMapper
and its descendants VmStaticRowMapper and VmTemplateRowMapper.
Also extracted fields found in both children back to
AbstractVmRowMapper.
Change-Id: Iab2a3bf9a0deb25b3950f75a351bde4b9e6fef74
Signed-off-by: Martin Betak <mbetak at redhat.com>
---
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/AbstractVmRowMapper.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/BaseDAODbFacade.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmStaticDAODbFacadeImpl.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmTemplateDAODbFacadeImpl.java
4 files changed, 12 insertions(+), 26 deletions(-)
Approvals:
Martin Peřina: Looks good to me, but someone else must approve
Martin Betak: Verified
Omer Frenkel: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/29518
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iab2a3bf9a0deb25b3950f75a351bde4b9e6fef74
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Betak <mbetak at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Martin Betak <mbetak at redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
More information about the Engine-commits
mailing list