Moti Asayag has posted comments on this change.
Change subject: core: Removing very bad code
......................................................................
Patch Set 1: (4 inline comments)
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VDSBrokerFrontendImpl.java
Line 47: if (!StringUtils.isEmpty(result.getExceptionString())) {
you can use StringUtils.isNotEmpty() and remove the negation.
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsEventListener.java
Line 201: if (!StringUtils.isEmpty(vmDynamic.getclient_ip())) {
same as previous comment
....................................................
File
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java
Line 770:
ResourceManager.getInstance().getEventListener().storagePoolStatusChanged(storagePool.getId(),
storagePool.getstatus());
please run formatter for this line. it shouldn't be that long
....................................................
File
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ResourceManager.java
Line 110: private final ConcurrentHashMap<Guid, Boolean> _asyncRunningVms =
shames there is no ConcurrentHashSet which encapsulates the usesless Boolean
usage.
--
To view, visit http://gerrit.ovirt.org/6097
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I5420f7446c8e2b0d082e2ca6f1a0c54432321fd0
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Michael Kublin <[email protected]>
Gerrit-Reviewer: Michael Kublin <[email protected]>
Gerrit-Reviewer: Moti Asayag <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
