Change in ovirt-engine[master]: core: Remove unused GetTimeLeasedUsersByVmPoolId

amureini at redhat.com amureini at redhat.com
Wed Mar 27 08:00:41 UTC 2013


Allon Mureinik has submitted this change and it was merged.

Change subject: core: Remove unused GetTimeLeasedUsersByVmPoolId
......................................................................


core: Remove unused GetTimeLeasedUsersByVmPoolId

Removed the unused GetTimeLeasedUsersByVmPoolId query.

This patch removes:
1. The GetTimeLeasedUsersByVmPoolIdQuery class
2. The GetTimeLeasedUsersByVmPoolIdParameters class only used by the
   aforementioned query.
3. The GetTimeLeasedUsersByVmPoolId constant from VdcQueryType
   representing the aforementioned query.
4. The getAllTimeLeasedUsersForVm(int) method from DbUserDAO, its
   implementation and its "test", as it was only used by the
   aforementioned query. Note that the underlying stored procedure no
   longer exists, so this DAO would not have worked anyway.

Change-Id: I7db6131e621dd1f34d023565d87c1a6afecbf2fa
Signed-off-by: Allon Mureinik <amureini at redhat.com>
---
D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetTimeLeasedUsersByVmPoolIdQuery.java
D backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/GetTimeLeasedUsersByVmPoolIdParameters.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/DbUserDAO.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/DbUserDAODbFacadeImpl.java
M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/DbUserDAOTest.java
6 files changed, 0 insertions(+), 59 deletions(-)

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7db6131e621dd1f34d023565d87c1a6afecbf2fa
Gerrit-PatchSet: 4
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: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list