Allon Mureinik has submitted this change and it was merged. Change subject: core: Remove IsVdsWithSameNameExist ......................................................................
core: Remove IsVdsWithSameNameExist IsVdsWithSameNameExistQuery is not used anywhere in the code, and hence should be removed. This patch removes the following: 1. The IsVdsWithSameNameExistQuery class 2. The IsVdsWithSameNameExist constant representing it in VdcQueryType 3. The IsVdsWithSameNameExistParameters class which was only used by the removed query class. 3. The getByVdsName(String) method in VdsStaticDAO (which was only used by the query), its implementation, and its test. 4. The GetVdsStaticByVdsNam stored procedure that was only used by the said DAO function. Change-Id: I676cca49a2cc62c7b0716589ee26d2bcffd3cbb7 Signed-off-by: Allon Mureinik <[email protected]> --- M backend/manager/dbscripts/vds_sp.sql D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/IsVdsWithSameNameExistQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsHandler.java D backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/IsVdsWithSameNameExistParameters.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsStaticDAO.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsStaticDAODbFacadeImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VdsStaticDAOTest.java 7 files changed, 0 insertions(+), 98 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13290 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I676cca49a2cc62c7b0716589ee26d2bcffd3cbb7 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: Omer Frenkel <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: Vered Volansky <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
