841721 - add releases_history index to balrog's model

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[[e-mail] (:bhearsum)]

Sheeri added an index to this table in bug 830346. We should get the index into the sqlalchemy model to make sure local tests are similar and that we don't forget about it completely.

Comment 1

[[e-mail] (:bhearsum)]

From bug 830346:
(In reply to Sheeri Cabral [:sheeri] from comment #20)
> Sure!
> 
> (I ran this on my tablet last night, otherwise I usually copy and paste
> stuff into the bug)
> 
> ALTER TABLE aus4_dev.releases_history add index(timestamp);
> 
> it took 27 minutes on dev2 and 28 minutes on dev1, just for my own
> information.
> 
> Also, the table rebuild showed the table had no (or very little)
> fragmentation:
> 
> [[e-mail]1 aus4_dev]# ls -lrth releases_history.*
> -rw-rw---- 1 mysql mysql 8.7K Feb 15 02:26 releases_history.frm
> -rw-rw---- 1 mysql mysql  55G Feb 15 02:47 releases_history.MYD
> -rw-rw---- 1 mysql mysql 7.7M Feb 15 02:55 releases_history.MYI

Comment 2

[[e-mail] (:bhearsum)]

Attachment: implement downgrades; add index

This index was added to the dev db by hand in bug 830346. I'll need to update the migrate_version in that db after this lands, otherwise we'll get a failure next time we try to do an upgrade to the schema through manage_db.py.

Comment 3

[[github robot]]

Commit pushed to master at https://github.com/mozilla/balrog

https://github.com/mozilla/balrog/commit/48b6165fc837312ab67989b5e3fd1d06ddcc20e7
bug 841721: add releases_history index to balrog's model. r=catlee

Comment 4

[[e-mail] (:bhearsum)]

Comment on attachment 714457 [details] [diff] [review]
implement downgrades; add index

Landed. Couldn't update the table yet because of bug 842552.

Comment 5

[[e-mail] (:bhearsum)]

Turns out that was user error. I got it updated:
mysql> select * from migrate_version;
+---------------+-----------------+---------+
| repository_id | repository_path | version |
+---------------+-----------------+---------+
| balrog        | auslib/migrate  |       3 |
+---------------+-----------------+---------+