Allon Mureinik has posted comments on this change.

Change subject: core: Quota refactor
......................................................................


Patch Set 11: I would prefer that you didn't submit this

(1 inline comment)

....................................................
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmCommand.java
Line 17: import org.ovirt.engine.core.bll.utils.VmDeviceUtils;
Line 18: import org.ovirt.engine.core.common.AuditLogType;
Line 19: import org.ovirt.engine.core.common.VdcObjectType;
Line 20: import org.ovirt.engine.core.common.action.VmManagementParametersBase;
Line 21: import org.ovirt.engine.core.common.businessentities.*;
please don't.
Line 22: import org.ovirt.engine.core.common.locks.LockingGroup;
Line 23: import 
org.ovirt.engine.core.common.queries.IsVmWithSameNameExistParameters;
Line 24: import org.ovirt.engine.core.common.queries.VdcQueryType;
Line 25: import org.ovirt.engine.core.common.utils.VmDeviceType;


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I109ae8ae0191f46abaa1be21a7ecfebe50b9993f
Gerrit-PatchSet: 11
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: ofri masad <oma...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchap...@redhat.com>
Gerrit-Reviewer: ofri masad <oma...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to