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 <[email protected]> --- 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 <[email protected]> Gerrit-Reviewer: Alissa Bonas <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Liron Ar <[email protected]> Gerrit-Reviewer: Maor Lipchuk <[email protected]> Gerrit-Reviewer: Sergey Gotliv <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: Vered Volansky <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
