Allon Mureinik has uploaded a new change for review.

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(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/63/3563/1
--
To view, visit http://gerrit.ovirt.org/3563
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib81f8a76d09de32dda62b75e01e70e536ab8df61
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to