Defect #1071
Wiki/News/Documents truncate contents to 64K in MYSQL
| Status: | Closed | Start date: | 2008-04-17 | |
|---|---|---|---|---|
| Priority: | Urgent | Due date: | ||
| Assignee: | Jean-Philippe Lang | % Done: | 0% | |
| Category: | Database | |||
| Target version: | 0.9.0 | |||
| Affected version: | 0.8.7 | Resolution: | Fixed |
Description
I've found wiki posts over 64K get silently truncated. The problem is in the SQL specification of the following databases/fields is TEXT (64K) instead of MEDIUMTEXT (16MB) or LONGTEXT (>16MB):
| table changesets | comments |
| table comments | comments |
| table documents | description |
| table issues | description |
| table journals | notes |
| table messages | content |
| table news | description |
| table projects | description |
| table wiki_contents | text |
Perhaps some of these don't need to change but I think most should be MEDIUMTEXT or LARGETEXT (documents for example)?
I have worked around this at my site with using the mysql command:
mysql> ALTER TABLE wiki_contents CHANGE text MEDIUMTEXT ;
but I'm afraid the next db migrate will mess it up, and others will hit it.
I suggest this is urgent priority, since data is lost.
Related issues
Associated revisions
Enlarge wiki content for MySQL databases (#1071).
Limit changed from 64KB to 16MB.
History
#1 Updated by W Snyder about 5 years ago
There is also a BLOB in wiki_content_versions that should be MEDIUMBLOB or LARGEBLOB.
#2 Updated by Thomas Lecavelier almost 5 years ago
Patch provided for it there: #1404
Hoping it helps.
#3 Updated by Lars Erik Gullerud over 3 years ago
#4 Updated by W Snyder over 3 years ago
Ping - could the patch or this get integrated? This seems a serious defect. Thanks.
#5 Updated by Jean-Philippe Lang over 3 years ago
- Category set to Database
#6 Updated by Thomas Lecavelier over 3 years ago
- File raise_wiki_content_size.patch
added - Assignee set to Jean-Philippe Lang
Rewrite from the patch #1404 : set wiki content and versioned content up to 16MB.
I can eventually add a wiki content truncation detection system, but it'd cost a reload of the modified instance. Is this patch ok or should I add the check?
#7 Updated by W Snyder over 3 years ago
IMO 16MB is fine. It becomes a HTML page not an attachment, so 16MB is enough.
#8 Updated by Prof. Dr. YoMan over 3 years ago
Will this go into 0.9?
I needed to split up some documents with big tables into two or more parts which isnt really nice.
#9 Updated by Jean-Philippe Lang over 3 years ago
- Status changed from New to Resolved
- Target version set to 0.9.0
- Affected version set to 0.8.7
- Resolution set to Fixed
Fixed in r3254.
A mysql specific migration was added to raise the limit.
#10 Updated by Jean-Philippe Lang over 3 years ago
- Status changed from Resolved to Closed
Merged in 0.9-stable in r3255.