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

amureini at redhat.com amureini at redhat.com
Sun Mar 31 05:51:28 UTC 2013


Allon Mureinik has submitted this change and it was merged.

Change subject: core: Remove GetAllIdsFromExportDomainQuery
......................................................................


core: Remove GetAllIdsFromExportDomainQuery

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

This patch removes the following:
1. The GetAllIdsFromExportDomainQuery class
2. The GetAllIdsFromExportDomain constant representing it in
   VdcQueryType
3. The GetVmsListVDSCommand class which was only used by
   this query.
4. The GetVmsList constant representing it in
   VDSCommandType.
5. The getVmsList(String, String)  method in
   IIrsServer/IrsServerWrapper, which was only used
   by GetVmsListVDSCommand.
6. The GetVmsListReturnForXmlRpc class which was only used by
   GetVmsListVDSCommand.

Change-Id: Id6c384ccbb317f67afb4d0fc13373dd6c7671f93
Signed-off-by: Allon Mureinik <amureini at redhat.com>
---
D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/GetAllIdsFromExportDomainQuery.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/irsbroker/GetVmsListReturnForXmlRpc.java
D backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/GetVmsListVDSCommand.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IIrsServer.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsServerWrapper.java
7 files changed, 0 insertions(+), 96 deletions(-)

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id6c384ccbb317f67afb4d0fc13373dd6c7671f93
Gerrit-PatchSet: 3
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