Shireesh Anjal has posted comments on this change.

Change subject: gluster: fixed NPE in GlusterAuditLogUtil
......................................................................


Patch Set 1: (1 inline comment)

Thanks, new patch-set to follow.

....................................................
File 
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/gluster/GlusterAuditLogUtil.java
Line 26:         return instance;
Line 27:     }
Line 28: 
Line 29:     public void logVolumeMessage(final GlusterVolumeEntity volume, 
final AuditLogType logType) {
Line 30:         logAuditMessage(volume.getClusterId(), volume, null, logType, 
new HashMap<String, String>());
Done
Line 31:     }
Line 32: 
Line 33:     public void logServerMessage(final VDS server, final AuditLogType 
logType) {
Line 34:         logAuditMessage(null, null, server, logType, new 
HashMap<String, String>());


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id3b563573670c379e085b641174cd7b1f0de2864
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shireesh Anjal <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: Sahina Bose <[email protected]>
Gerrit-Reviewer: Shireesh Anjal <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to