sopel icon indicating copy to clipboard operation
sopel copied to clipboard

value.nick_values column isn't long enough

Open RhinosF1 opened this issue 3 years ago • 2 comments

Description

sopel/db.py#L66 states it should be 255 charecters. That doesn't even fit seen which is from core as that needs at least 512. This should be made higher.

Reproduction steps

  1. Run bot with mysql/MariaDB
  2. Say something long3.
  3. See error4.

Expected behavior

Column is big enough

Logs

None yet, I can post if really needed

Environment

  • Sopel .version: somewhere between 7 & 8
  • Sopel installed via: source
  • Python version: 3.9.2
  • Operating system: Bullseye
  • IRCd /version: Libera
  • Relevant plugins: core, seen

Notes

TEXT might be good, moaned on IRC. Will probably make TEXT.

RhinosF1 avatar May 29 '22 17:05 RhinosF1

MariaDB [sopelbeta]> ALTER TABLE nick_values MODIFY value TEXT; Stage: 1 of 2 'copy to tmp table' 16.8% of stage doStage: 2 of 2 'Enabling keys' 0% of stage done Query OK, 5947 rows affected (0.496 sec)
Records: 5947 Duplicates: 0 Warnings: 0

RhinosF1 avatar May 29 '22 17:05 RhinosF1

This should be handled via #2317

RustyBower avatar Jul 13 '22 00:07 RustyBower