Change in ovirt-engine[master]: core: Introducing saveOrUpdate to AsyncTasks

yzaslavs at redhat.com yzaslavs at redhat.com
Wed Nov 14 12:17:00 UTC 2012


Yair Zaslavsky has submitted this change and it was merged.

Change subject: core: Introducing saveOrUpdate to AsyncTasks
......................................................................


core: Introducing saveOrUpdate to AsyncTasks

This performance improvement to the add or update of tasks is introduced -
No more querying to check the db if tasks exists and
according to it deciding if to save or update, but
performing all this logic at DB side.

Bug-Url: https://bugzilla.redhat.com/873546
Change-Id: Iebfc85569ba1aa8bd840f7239f83b7f921a4bd7e
Signed-off-by: Yair Zaslavsky <yzaslavs at redhat.com>
---
M backend/manager/dbscripts/async_tasks_sp.sql
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/AsyncTaskUtils.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/AsyncTaskDAO.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/AsyncTaskDAODbFacadeImpl.java
M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/AsyncTaskDAOTest.java
5 files changed, 84 insertions(+), 7 deletions(-)

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iebfc85569ba1aa8bd840f7239f83b7f921a4bd7e
Gerrit-PatchSet: 3
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: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list