Moti Asayag has posted comments on this change. Change subject: core: ResourceManager#InternalSetVmStatus cleanup ......................................................................
Patch Set 1: (1 inline comment) .................................................... File backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ResourceManager.java Line 5: import java.util.List; Line 6: import java.util.Map; Line 7: import java.util.concurrent.ConcurrentHashMap; Line 8: Line 9: import org.apache.commons.lang.StringUtils; This new import isn't being used. This patch by its own will cause checkstyle to break the build. This import should be removed from this patch and introduced if necessary in the relevant patch. Line 10: import org.ovirt.engine.core.common.AuditLogType; Line 11: import org.ovirt.engine.core.common.businessentities.IVdsEventListener; Line 12: import org.ovirt.engine.core.common.businessentities.VDS; Line 13: import org.ovirt.engine.core.common.businessentities.VDSStatus; -- To view, visit http://gerrit.ovirt.org/10617 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ie65daceebb3d34d36e8baf8e1c38e3a669f73e7d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <[email protected]> Gerrit-Reviewer: Arik Hadas <[email protected]> Gerrit-Reviewer: Moti Asayag <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Roy Golan <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
