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(a)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(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Liran Zelkha <liran.zelkha(a)gmail.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server