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 <[email protected]> --- 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 <[email protected]> Gerrit-Reviewer: Eli Mesika <[email protected]> Gerrit-Reviewer: Liran Zelkha <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
