1201140 - Add existing indexes to the corresponding orm model fields

Status: RESOLVED FIXED

(Tree Management :: Treeherder, defect)

Description

[Mauro Doglio [:mdoglio]]

To completely replace the reference data sql template files with orm-generated code we need make it aware of the existing indexes. This should as easy as adding either a index=True on the corresponding field or via the index_together Meta attribute

Comment 1

[Mauro Doglio [:mdoglio]]

Attachment: PR 960

Comment 2

[Ed Morley [:emorley]]

Comment on attachment 8660801 [details] [review]
PR 960

Travis is failing.

Comment 3

[Mauro Doglio [:mdoglio]]

Comment on attachment 8660801 [details] [review]
PR 960

all fixed :-)

Comment 4

[Ed Morley [:emorley]]

Comment on attachment 8660801 [details] [review]
PR 960

Looks good, ty :-)

Comment 5

[Treeherder GitHub Bugbot]

Commits pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/aa91f022ab34a3123d28537f5bc8c6ee1728f74c
Bug 1201140 - Add sqlparse to the requirements file

This is required to execute migrations using migrations.RunSQL.
See https://docs.djangoproject.com/en/1.8/ref/migration-operations/#runsql for more info

https://github.com/mozilla/treeherder/commit/cbe1ab610eec648a26320aaf8d7eaf66be17f3ec
Bug 1201140 - Add key indexes to the ref data orm classes

Key indexes are added in both the orm classes and the initial migration.
For the fulltext indexes that wasn't possible, since The django orm doesn't support it.
I used a RunSQL operation to add them via sql.