From b0c4e551fd03704fd1fad3b14d8533c675cbe6ae Mon Sep 17 00:00:00 2001 From: Matthias Mullie Date: Wed, 5 Feb 2014 14:21:48 -0800 Subject: [PATCH] Add some more clarification on why event_extra can't be too large Change-Id: I3b366e06bb1980d834cad09c24dde929981ea366 --- model/Event.php | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/model/Event.php b/model/Event.php index a24764a46..60702810b 100644 --- a/model/Event.php +++ b/model/Event.php @@ -109,7 +109,9 @@ class EchoEvent { } // If the extra size is more than 50000 bytes, that means there is - // probably a problem with the design of this notification type + // probably a problem with the design of this notification type. + // There might be data loss if the size exceeds the DB column size of + // event_extra. if ( strlen( $obj->serializeExtra() ) > 50000 ) { wfDebugLog( __CLASS__, __FUNCTION__ . ': event extra data is too huge for ' . $info['type'] ); return false;