Change in ovirt-engine[master]: core: Remove GetStorageSessionsListQuery

amureini at redhat.com amureini at redhat.com
Thu Apr 4 09:16:59 UTC 2013


Allon Mureinik has submitted this change and it was merged.

Change subject: core: Remove GetStorageSessionsListQuery
......................................................................


core: Remove GetStorageSessionsListQuery

GetStorageSessionsListQuery is not used anywhere in the code, and
hence should be removed.

This patch removes the following:
1. The GetStorageSessionsListQuery class
2. The GetStorageSessionsList constant representing it in
   VdcQueryType
3. The GetStorageSessionsListVDSCommand class which was only used by
   this query.
4. The GetStorageSessionList constant representing it in
   VDSCommandType.
5. The getSessionsList() method
   IVdsServer/VdsServerConnector/VdsServerWrapper, which was only used
   by GetStorageSessionsListVDSCommand.
6. The SessionsListReturnForXmlRpc class which was only used by
   GetStorageSessionsListVDSCommand.

Change-Id: I5ee43140a7ca0cbe9a2c15cc787c270ead619bda
Signed-off-by: Allon Mureinik <amureini at redhat.com>
---
D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/GetStorageSessionsListQuery.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryType.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/VDSCommandType.java
D backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/GetStorageSessionsListVDSCommand.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/IVdsServer.java
D backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/SessionsListReturnForXmlRpc.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerConnector.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerWrapper.java
8 files changed, 0 insertions(+), 115 deletions(-)

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I5ee43140a7ca0cbe9a2c15cc787c270ead619bda
Gerrit-PatchSet: 6
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: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk at redhat.com>



More information about the Engine-commits mailing list