Mike Kolesnik has submitted this change and it was merged. Change subject: core: Remove VmPoolDAO.getAllVmPoolsForAdGroup ......................................................................
core: Remove VmPoolDAO.getAllVmPoolsForAdGroup Removed VmPoolDAO.getAllVmPoolsForAdGroup: On the one hand, it's based on a stored procedure that doesn't (and can't) work. On the other hand, it's only used by GetVmPoolsAttachedToAdGroupQuery, which isn't used anywhere on it's own right. This patch removes this entire stack of dead code - the stored procedure, the DAO, and the query that uses it. Change-Id: Ib81f8a76d09de32dda62b75e01e70e536ab8df61 --- M backend/manager/dbscripts/vm_pools_sp.sql D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetVmPoolsAttachedToAdGroupQuery.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryType.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmPoolDAO.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmPoolDAODbFacadeImpl.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmPoolDAOHibernateImpl.java 6 files changed, 0 insertions(+), 56 deletions(-) Approvals: Mike Kolesnik: Looks good to me, approved Allon Mureinik: Verified -- To view, visit http://gerrit.ovirt.org/3563 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib81f8a76d09de32dda62b75e01e70e536ab8df61 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Mike Kolesnik <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
