Change in ovirt-engine[master]: core: Remove StoragePoolIsoMapDAO.update impl

amureini at redhat.com amureini at redhat.com
Tue Apr 30 16:58:27 UTC 2013


Allon Mureinik has submitted this change and it was merged.

Change subject: core: Remove StoragePoolIsoMapDAO.update impl
......................................................................


core: Remove StoragePoolIsoMapDAO.update impl

Since the owner column was removed from StoragePoolIsoMap in the
previous patch, it's only (database) mutable column is the status
column.
In other words, there is no difference between the update and
updateStatus methods.

This patch removes the Updatestorage_pool_iso_map stored procedure
and replaces the implementation of the update method with a
delegation to updateStatus.

Change-Id: Ia661353ed7b100795d90c9390866c955a1a6290d
Signed-off-by: Allon Mureinik <amureini at redhat.com>
---
M backend/manager/dbscripts/storages_san_sp.sql
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/StoragePoolIsoMapDAODbFacadeImpl.java
2 files changed, 1 insertion(+), 23 deletions(-)

Approvals:
  Allon Mureinik: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia661353ed7b100795d90c9390866c955a1a6290d
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Alissa Bonas <abonas at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Liron Ar <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgotliv at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk at redhat.com>



More information about the Engine-commits mailing list