Eli Mesika has submitted this change and it was merged. Change subject: core: Fix AuditLog check for existing log entries ......................................................................
core: Fix AuditLog check for existing log entries The audit log check for existing element in the cache, If the entry exist, then there is no need for another log message. This is how the anti flooding mechanism works. For that we use Infinispan cache and the putIfAbsent method, which updates the lifespan(time to keep in cache). For our use of the cache, we don't want this update. The fix changes the putIfAbsent and replace it with two othe API methods, Contains and Put. Change-Id: I94b777911eb1ff667be3dc8362bde5746d84824c Bug-Url: https://bugzilla.redhat.com/1079920 Signed-off-by: Kobi Ianko <[email protected]> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/TimeoutBase.java 1 file changed, 11 insertions(+), 6 deletions(-) Approvals: Eli Mesika: Looks good to me, approved Kobi Ianko: Verified Yair Zaslavsky: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/26028 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I94b777911eb1ff667be3dc8362bde5746d84824c Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kobi Ianko <[email protected]> Gerrit-Reviewer: Doron Fediuck <[email protected]> Gerrit-Reviewer: Eli Mesika <[email protected]> Gerrit-Reviewer: Gilad Chaplik <[email protected]> Gerrit-Reviewer: Jiří Moskovčák <[email protected]> Gerrit-Reviewer: Kobi Ianko <[email protected]> Gerrit-Reviewer: Liran Zelkha <[email protected]> Gerrit-Reviewer: Martin Sivák <[email protected]> Gerrit-Reviewer: Moti Asayag <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
