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