Change in ovirt-engine[master]: Core: Changing EntityMapSqlMapper

yzaslavs at redhat.com yzaslavs at redhat.com
Sun Jun 30 02:40:56 UTC 2013


Yair Zaslavsky has submitted this change and it was merged.

Change subject: Core: Changing EntityMapSqlMapper
......................................................................


Core: Changing EntityMapSqlMapper

Changing this class to support mapping of objects
of any class.
The motivation is that we might to run batch jobs on constructs
that are not business entities (for example - a touple of
Guid,Guid,VdcObjectType to mark AsyncTask entity association -
I prefer not to have redundant code for creation business
entity for that)
In addition, the code for using the mapper was moved to the
CallsHandler, as MassOPerationsDao relies on a generic type
which extends BusinessEntity

Change-Id: I06291792ab70928de39fc2d12c40d4b2d247a2eb
Signed-off-by: Yair Zaslavsky <yzaslavs at redhat.com>
---
D backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/EntityToMapSqlParameterMapper.java
A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/MapSqlParameterMapper.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/SimpleJdbcCallsHandler.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/MassOperationsDao.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/MassOperationsGenericDaoDbFacade.java
5 files changed, 63 insertions(+), 46 deletions(-)

Approvals:
  Yair Zaslavsky: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I06291792ab70928de39fc2d12c40d4b2d247a2eb
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Liran Zelkha <liran.zelkha at gmail.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list