1202626 - Stop storing each revision's commit_timestamp

Status: RESOLVED FIXED

(Tree Management :: Treeherder, defect, P3)

Description

[Ed Morley [:emorley]]

At the moment we store both the resultset's time (as push_timestamp) and the commit timestamp for each revision (as commit_timestamp).

The latter is not surfaced in the UI anywhere, and IMO really isn't something we need - we're only ever thinking in terms of whole pushes - the only time the individual revisions are useful is when displaying the list of SHAs+authors in the UI. However the date adds little value there (it is typically the author date which can be in the past and has little relation to the date the commit landed - which is already stored as push_timestamp).

Let's just remove it.

Comment 1

[Ed Morley [:emorley]]

Attachment: Stop storing each revision's commit_timestamp

Comment 2

[Treeherder GitHub Bugbot]

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

https://github.com/mozilla/treeherder/commit/ba39c25fa20609e2845d9dcfc19f15b159411a28
Bug 1202626 - Stop storing each revision's commit_timestamp

It's unused in the UI and doesn't add any value, since we're
normally much more interested in the push_timestamp.

This can land without causing errors, since the field is DEFAULT NULL,
so can be dropped at our leisure later.