From 69fe833f6e2a5248059b0494e1cbf7c20642d4c0 Mon Sep 17 00:00:00 2001 From: Sijis Aviles Date: Fri, 19 Jun 2026 01:25:10 -0500 Subject: [PATCH] refactor: change column `value` from String to Text Changes the `value` Column data type in sql.py from String(32768) to Text(32768). This allows storing larger serialized JSON/jsonpickle strings and prevents row-size limit errors on database backends like MySQL when utilizing multi-byte (e.g., utf8mb4) character sets. --- sql.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sql.py b/sql.py index 60c2b6a..c6a5341 100644 --- a/sql.py +++ b/sql.py @@ -6,7 +6,7 @@ from errbot.storage.base import StorageBase, StoragePluginBase from jsonpickle import decode, encode -from sqlalchemy import Column, MetaData, String, Table, create_engine +from sqlalchemy import Column, MetaData, String, Table, Text, create_engine from sqlalchemy.orm import registry, sessionmaker from sqlalchemy.orm.exc import NoResultFound @@ -120,7 +120,7 @@ def open(self, namespace: str) -> StorageBase: namespace, self._metadata, Column("key", String(767), primary_key=True), - Column("value", String(32768)), + Column("value", Text(32768)), extend_existing=True, )