Edgewall Software
Modify

Opened 15 years ago

Closed 9 years ago

Last modified 9 years ago

#8396 closed enhancement (fixed)

[Database schema] Ticket description field might be too short.

Reported by: guillaume.brocard@… Owned by: Jun Omae
Priority: normal Milestone: 1.1.4
Component: database backend Version: 0.11.4
Severity: normal Keywords: mysql
Cc: Branch:
Release Notes:

Use mediumtext type for text columns on MySQL database since text type is too short.

API Changes:
Internal Changes:

Description

He,

have you ever considered using mediumtext instead of text for the description field of a ticket ?

I've got tickets exceeding the 65536 characters limit, that have been tested ok when switching to mediumtext type (in my case, this is useful for an import from clearquest tickets).

I have checked Bugzilla, and they are using mediumtext pretty much everywhere (at least in version 3.2).

Attached is the modified schema file.

Thx, Guillaume.


System information :

os : windows xp 32 bits sp2
py : 2.5
db : mysql server 5.0

Attachments (1)

db_default.py (13.1 KB ) - added by guillaume.brocard@… 15 years ago.
Default db schema definition, modified under "Ticket system" for the description column type.

Download all attachments as: .zip

Change History (12)

by guillaume.brocard@…, 15 years ago

Attachment: db_default.py added

Default db schema definition, modified under "Ticket system" for the description column type.

comment:1 by Christian Boos, 15 years ago

Keywords: mysql added; patch removed

Please, take the time to produce and submit a real patch.

Also, be sure to review related tickets, in particular #6986.

comment:2 by Christian Boos, 15 years ago

Component: ticket systemdatabase backend
Milestone: 1.0
Severity: trivialnormal

comment:3 by Christian Boos, 14 years ago

Milestone: 1.0unscheduled

Milestone 1.0 deleted

comment:4 by Christian Boos, 14 years ago

Milestone: triagingnext-major-0.1X

Related to the distinction of string vs. text in GenericTrac.

comment:5 by Christian Boos, 11 years ago

See also #11032 (closed as duplicate) which reported a similar issue but with wiki pages.

comment:6 by Jun Omae, 9 years ago

Milestone: next-major-releases1.1.4
Owner: set to Jun Omae
Status: newassigned

Same issue on our environments. I think we should use mediumtext type rather than text type for MySQL.

In jomae.git@t8396, proposed changes adds db36.py which changes text type to mediumtext type using directly ALTER TABLE.

comment:7 by Jun Omae, 9 years ago

Rebased jomae.git@t8396 which adds db37.py. I'll push it tomorrow.

comment:8 by Jun Omae, 9 years ago

Release Notes: modified (diff)
Resolution: fixed
Status: assignedclosed

Committed in [13767].

comment:9 by Christian Boos, 9 years ago

Just curious, did you measure the impact on performance and on the size of the DB (before/after upgrade)?

comment:10 by Jun Omae, 9 years ago

Yeah. Very few differences on the performance and the size between text and mediumtext types.

comment:11 by Ryan J Ollos, 9 years ago

#11285 closed as a duplicate.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Jun Omae.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Jun Omae to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.