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)

enhancement

Tracking

(Not tracked)

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.
Assignee: nobody → ghickman
Attachment #8987001 - Flags: review?(emorley)
Attachment #8987002 - Flags: review?(emorley)
Attachment #8987001 - Flags: review?(emorley) → review+
Comment on attachment 8987002 [details] 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)
Assignee: ghickman → nobody
Priority: -- → P3
Blocks: 1178227
Component: Treeherder → Database
Component: Database → TreeHerder
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: