Laszlo Hornyak has submitted this change and it was merged.
Change subject: core: Remove GetAvailableClustersByServerCpu
......................................................................
core: Remove GetAvailableClustersByServerCpu
GetAvailableClustersByServerCpuQuery is not used anywhere in
the code, and hence should be removed.
This patch removes the following:
1. The GetAvailableClustersByServerCpuQuery class.
2. The GetAvailableClustersByServerCpu constant representing
it in VdcQueryType.
3. The GetAvailableClustersByServerCpuParameters class only used by
the aforementioned query.
4. Methods in CpuFlagsManagerHandler only used by the
aforementioned query: GetAllServerCpusBelowCpu(String, Version)
and GetAllServerCpusBelowCpu(String).
Change-Id: Id8bca65412e18c2c62ead41825b08cddd9506c11
Signed-off-by: Allon Mureinik <amureini(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CpuFlagsManagerHandler.java
D
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetAvailableClustersByServerCpuQuery.java
D
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/GetAvailableClustersByServerCpuParameters.java
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryType.java
4 files changed, 0 insertions(+), 100 deletions(-)
Approvals:
Laszlo Hornyak: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/13600
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id8bca65412e18c2c62ead41825b08cddd9506c11
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>