Open
Bug 287311
Opened 20 years ago
Updated 11 years ago
Change old enum fields from varchars to integers with foreign keys
Categories
(Bugzilla :: Database, enhancement)
Tracking
()
NEW
People
(Reporter: mkanat, Unassigned)
References
(Depends on 6 open bugs, Blocks 3 open bugs)
Details
(Keywords: meta)
I moved the old "enum" fields to be varchars, because that happened to be the
easiest thing to do to get rid of enums.
Now, they need to be changed to be integers in the bugs table, which point to
the correct varchar value in a separate table.
This is a meta bug, we'll move the fields one at a time, each in a separate bug.
Reporter | ||
Comment 1•20 years ago
|
||
Oh, by the way, as these changes are being made, they should have no UI impact.
That is, they should be entirely architectural changes. Any UI changes can
happen later.
Reporter | ||
Comment 2•20 years ago
|
||
Ahhh, crap! There are no midairs on dep changes. :-( Adding all the deps again.
Sorry about that.
Comment 3•17 years ago
|
||
Please note that this re-normalization does have a penalty to reporting... Not a big deal, but it is what it is...
Updated•17 years ago
|
Assignee: general → kevin.benton
Comment 5•17 years ago
|
||
It seems that removing bugs.keywords is the right way to go with this too. I know that mattyt proposed that the column should be smaller, but I propose going a step further - remove the field and just use the bug_keyword_map table instead. bugs.keywords just contains information that's duplicated in the map anyway.
Depends on: 86545
Comment 6•17 years ago
|
||
Sorry all - wrong bug. Just realized that 86545 is about buglist.cgi not removing the keywords column in buglist.cgi
No longer depends on: 86545
Comment 7•17 years ago
|
||
I am unable to work this bug at this time due to other priorities. Reassigning to default assignee.
Assignee: kevin.benton → general
Updated•11 years ago
|
Assignee: general → database
Component: Bugzilla-General → Database
You need to log in
before you can comment on or make changes to this bug.
Description
•