Attachment 37662 Details for Bug 13606 – Bug 13606: Upgrade sessions.a_session to MEDIUMTEXT

[patch] Bug 13606: Upgrade sessions.a_session to MEDIUMTEXT

Bug-13606-Upgrade-sessionsasession-to-MEDIUMTEXT.patch (text/plain), 2.25 KB, created by Mark Tompsett on 2015-04-10 16:22:09 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Mark Tompsett

Created: 2015-04-10 16:22:09 UTC

Size: 2.25 KB

patch

obsolete

>From bf70f279ce4d9e9ba3657e303e5ea883f34decff Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@biblibre.com>
>Date: Wed, 21 Jan 2015 15:59:10 +0100
>Subject: [PATCH] Bug 13606: Upgrade sessions.a_session to MEDIUMTEXT
>
>The previous patch fixed partially the problem.
>If you try to update more that 1000 records, you will get the same
>error.
>
>This patch change the type of sessions.a_session from TEXT (64kb) MEDIUMTEXT (32mb).
>
>Test plan:
>1/ Try to modify a huge amount of records (1000 should be enough)
>2/ Confirm it worked!
>
>Signed-off-by: Frederic Demians <f.demians@tamil.fr>
>
>I confirm that with more than 1000 records the session object is truncated, and
>then can't be anymore be desirialized, blocking completely Koha. Fixed with
>this patch.
>
>Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
>
>This only pushes off the problem expontentially (64KB vs. 16MB), but this is a
>good start.
>---
> installer/data/mysql/kohastructure.sql | 2 +-
> installer/data/mysql/updatedatabase.pl | 8 ++++++++
> 2 files changed, 9 insertions(+), 1 deletion(-)
>
>diff --git a/installer/data/mysql/kohastructure.sql b/installer/data/mysql/kohastructure.sql
>index 6ddba43..9786895 100644
>--- a/installer/data/mysql/kohastructure.sql
>+++ b/installer/data/mysql/kohastructure.sql
>@@ -1982,7 +1982,7 @@ CREATE TABLE `serial` ( -- issues related to subscriptions
> DROP TABLE IF EXISTS sessions;
> CREATE TABLE sessions (
>   `id` varchar(32) NOT NULL,
>-  `a_session` text NOT NULL,
>+  `a_session` mediumtext NOT NULL,
>   PRIMARY KEY (`id`)
> ) ENGINE=InnoDB DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci;
> 
>diff --git a/installer/data/mysql/updatedatabase.pl b/installer/data/mysql/updatedatabase.pl
>index 78e79d3..89b8705 100755
>--- a/installer/data/mysql/updatedatabase.pl
>+++ b/installer/data/mysql/updatedatabase.pl
>@@ -9978,6 +9978,14 @@ if ( CheckVersion($DBversion) ) {
>     SetVersion($DBversion);
> }
> 
>+$DBversion = "3.19.00.XXX";
>+if ( CheckVersion($DBversion) ) {
>+    $dbh->do(q|
>+         ALTER TABLE sessions CHANGE COLUMN a_session a_session MEDIUMTEXT
>+    |);
>+    print "Upgrade to $DBversion done (Bug 13606: Upgrade sessions.a_session to MEDIUMTEXT)\n";
>+    SetVersion($DBversion);
>+}
> 
> # DEVELOPER PROCESS, search for anything to execute in the db_update directory
> # SEE bug 13068
>-- 
>1.9.1

Actions: View | Diff | Splinter Review

Attachments on bug 13606: 35444 | 35445 | 37597 | 37601 | 37621 | 37627 | 37633 | 37634 | 37635 | 37636 | 37637 | 37648 | 37658 | 37660 | 37662 | 37663 | 37664 | 38058 | 38059 | 38060 | 38061 | 38431