Alon Bar-Lev has posted comments on this change. Change subject: engine : Remove StopVmParameters from StopVmCommand ......................................................................
Patch Set 14: (1 comment) .................................................... File backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/VmVdcParameter.java Line 2: Line 3: import org.ovirt.engine.core.common.action.StopVmTypeEnum; Line 4: import org.ovirt.engine.core.compat.Guid; Line 5: Line 6: public class VmVdcParameter { I am thinking of reducing the number of these classes... into: 1. core 2. infra 3. virt 4. storage 5. network 6. sla etc... Line 7: Line 8: public static final VdcParameter StopVmType = VdcParameter.create("StopVmType", StopVmTypeEnum.class, "ae6f0065-cae3-41ac-a3d9-2fd8edd4ad90"); Line 9: public static final VdcParameter VmId = VdcParameter.create("VmId", Guid.class, "b06b7c0f-320b-44c2-b114-408e4046645b"); Line 10: -- To view, visit http://gerrit.ovirt.org/20479 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I404816e3e70ce044e1348af0eeea18b59e521581 Gerrit-PatchSet: 14 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rn...@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Liran Zelkha <lzel...@redhat.com> Gerrit-Reviewer: Ravi Nori <rn...@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches