Michael Kublin has submitted this change and it was merged. Change subject: engine: Additional clean up in AuditLogDirector ......................................................................
engine: Additional clean up in AuditLogDirector Variable auditLogable can not be null, if it null we will fail on NullPointerException inside updateTimeoutLogableObject method. After removing such condition, were removed all relevant and dead code. Also, I chnaged tempVar to auditLog varibale, don't see reason for using tempVar Change-Id: Ia0eb406d78965bb0d84c7ce80761028877ab2879 Signed-off-by: Michael Kublin <[email protected]> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogDirector.java 1 file changed, 38 insertions(+), 48 deletions(-) Approvals: Michael Kublin: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13426 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia0eb406d78965bb0d84c7ce80761028877ab2879 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <[email protected]> Gerrit-Reviewer: Barak Azulay <[email protected]> Gerrit-Reviewer: Eli Mesika <[email protected]> Gerrit-Reviewer: Michael Kublin <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
