Change in ovirt-engine[master]: core: Removed EndedTaskInfo.commandShouldBeLogged

amureini at redhat.com amureini at redhat.com
Thu Dec 5 14:06:26 UTC 2013


Allon Mureinik has submitted this change and it was merged.

Change subject: core: Removed EndedTaskInfo.commandShouldBeLogged
......................................................................


core: Removed EndedTaskInfo.commandShouldBeLogged

This member is initialized to true, never modified and never queried,
and thus is redundant.

This patch removes the data member, its getter and setter, and the
single default constructor which did nothing but initialize this member.

Change-Id: Idac0e8c7cddf1eb797c856399b08a1cc6c8507e7
Signed-off-by: Allon Mureinik <amureini at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/asynctasks/EndedTasksInfo.java
1 file changed, 0 insertions(+), 14 deletions(-)

Approvals:
  Allon Mureinik: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Idac0e8c7cddf1eb797c856399b08a1cc6c8507e7
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Alissa Bonas <abonas at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list