Shireesh Anjal has submitted this change and it was merged. Change subject: gluster: improved gluster audit log util ...................................................................... gluster: improved gluster audit log util Create object of AuditLogableBase and set appropriate properties on it instead of overriding methods. Refactoring in AuditLogableBase: - Set vds id inside setVds - New setter for glusterVolume Change-Id: I52212ddb694d727248182d32abc1e1f5a21f76ee Bug-Url: https://bugzilla.redhat.com/916112 Signed-off-by: Shireesh Anjal <sanjal@redhat.com> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogableBase.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/gluster/GlusterAuditLogUtil.java 2 files changed, 17 insertions(+), 32 deletions(-) Approvals: Shireesh Anjal: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13001 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I52212ddb694d727248182d32abc1e1f5a21f76ee Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shireesh Anjal <sanjal@redhat.com> Gerrit-Reviewer: Eli Mesika <emesika@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel@redhat.com> Gerrit-Reviewer: Sahina Bose <sabose@redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs@redhat.com>