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