Ravi Nori has posted comments on this change.
Change subject: engine : Change parameter type of CommandBase
......................................................................
Patch Set 3:
(2 comments)
....................................................
File
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionParametersBase.java
Line 232: }
Line 233:
Line 234: @Override
Line 235: public int hashCode() {
Line 236: return super.hashCode();
You are right we can remove both equals and hashCode
Line 237: }
Line 238:
Line 239: @Override
Line 240: public boolean equals(Object obj) {
....................................................
File
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/VdcParameterHelper.java
Line 2:
Line 3: import java.util.List;
Line 4: import org.ovirt.engine.core.common.action.VdcActionParametersBase;
Line 5:
Line 6: public class VdcParameterHelper {
Rename to CommandUtils?
Line 7:
Line 8: public static boolean isTaskGroupSuccess(VdcParametersMap
parameters) {
Line 9: boolean childrenTasksSuccess =
parameters.<Boolean>get(CoreVdcParameters.TASK_GROUP_SUCCESS);
Line 10: List<VdcActionParametersBase> imagesParameters =
parameters.<List<VdcActionParametersBase>>get(CoreVdcParameters.IMAGE_PARAMETERS);
--
To view, visit http://gerrit.ovirt.org/21319
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I5c702d8f4184f08690d38902593bb4b733c69d95
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@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
