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 <[email protected]> --- 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 <[email protected]> Gerrit-Reviewer: Eli Mesika <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
