Change old enum fields from varchars to integers with foreign keys

NEW
Unassigned

Status

()

--
enhancement
14 years ago
5 years ago

People

(Reporter: mkanat, Unassigned)

Tracking

(Depends on: 6 bugs, Blocks: 3 bugs, {meta})

2.19.2
Dependency tree / graph

Details

(Reporter)

Description

14 years ago
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)

Updated

14 years ago
Depends on: 287314
(Reporter)

Updated

14 years ago
Depends on: 287316
(Reporter)

Updated

14 years ago
Depends on: 287317
(Reporter)

Updated

14 years ago
Depends on: 287318
(Reporter)

Updated

14 years ago
Depends on: 287320
(Reporter)

Updated

14 years ago
Depends on: 287321
(Reporter)

Comment 1

14 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.
Blocks: 278579
No longer depends on: 287316, 287317, 287318, 287320, 287321
(Reporter)

Comment 2

14 years ago
Ahhh, crap! There are no midairs on dep changes. :-( Adding all the deps again.
Sorry about that.
Depends on: 287316, 287317, 287318, 287320, 287321
(Reporter)

Updated

14 years ago
Blocks: 287322

Comment 3

11 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

11 years ago
Duplicate of this bug: 398734

Updated

11 years ago
Assignee: general → kevin.benton

Comment 5

11 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

11 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

11 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

10 years ago
Blocks: 462066

Updated

5 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.