Change in ovirt-engine[master]: core: moving AuditLog message length check to DB

emesika at redhat.com emesika at redhat.com
Mon Dec 31 11:50:14 UTC 2012


Eli Mesika has submitted this change and it was merged.

Change subject: core: moving AuditLog message length check to DB
......................................................................


core: moving AuditLog message length check to DB

The check in the setMessage breaks API tests since the created mock has
no access to configuration.
Since this check is relevant only for External Events, it was moved to
the stored procedure.

Change-Id: Ib2805ee22301d75a7d6a2c852a3acfe3fc0f5f59
Signed-off-by: Eli Mesika <emesika at redhat.com>
---
M backend/manager/dbscripts/audit_log_sp.sql
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/AuditLog.java
2 files changed, 9 insertions(+), 10 deletions(-)

Approvals:
  Eli Mesika: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib2805ee22301d75a7d6a2c852a3acfe3fc0f5f59
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list