Open
Bug 1470381
Opened 6 years ago
Updated 3 years ago
Remove FailureLine's best_classification and best_is_verified fields
Categories
(Tree Management :: Treeherder, enhancement, P3)
Tree Management
Treeherder
Tracking
(Not tracked)
NEW
People
(Reporter: ghickman, Unassigned)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
We used to duplicate the data in these fields between them and TextLogErrorMetadata's versions. A recent change moved us to only using TextLogErrorMetadata's version so these can be dropped.
Django's migrations naively try to remove them the columns with two ALTER TABLE statements so we'll need to both remove the FK constraint from best_classification then create an SQL migration to drop the columns at the same time.
|
Reporter | |
Updated•6 years ago
|
Assignee: nobody → ghickman
|
|
Reporter | |
Comment 1•6 years ago
|
||
Attachment #8987001 -
Flags: review?(emorley)
|
|
Reporter | |
Comment 2•6 years ago
|
||
Attachment #8987002 -
Flags: review?(emorley)
|
||
Updated•6 years ago
|
Attachment #8987001 -
Flags: review?(emorley) → review+
|
|
||
Comment 3•6 years ago
|
||
Comment on attachment 8987002 [details] [review]
Remove FailureLine's best_classification and best_is_verified
Cancelling pending https://github.com/mozilla/treeherder/pull/3687#pullrequestreview-131179194
Attachment #8987002 -
Flags: review?(emorley)
|
|
||
Updated•6 years ago
|
Assignee: ghickman → nobody
|
|
||
Updated•6 years ago
|
Priority: -- → P3
|
||
Updated•5 years ago
|
Component: Treeherder → Database
|
Assignee | |
Updated•3 years ago
|
Component: Database → TreeHerder
You need to log in
before you can comment on or make changes to this bug.
Description
•