Closed Bug 1196804 Opened 9 years ago Closed 9 years ago

Add last_modified field to result_set table

Categories

(Tree Management :: Treeherder: Data Ingestion, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: camd, Assigned: camd)

References

Details

Attachments

(1 file)

(deleted), text/x-github-pull-request
emorley
: review+
Details
This will be much like we have for the ``jobs`` table.
Assignee: nobody → cdawson
Chatted in the meeting with emorley that perhaps this won't be necessary. Though, on further thought, I still think it is. Here's why: If we get a pulse message for a push id of 100, then get one for pushID 102, we will assume we missed 101. But our steps will be this: 1. Write the 102 we just got 2. schedule to fetch 101. 3. create 101 when we get it. Between step 2 and 3, a user may have fetched resultsets and gotten only 100 and 102, but not 101. When 101 comes in, the UI won't know to look for it. Fetching resultsets based on last_modified would fix this problem.
Blocks: 1169320
No longer blocks: 1169320
Blocks: 1199511
I will also need fields ``revision`` and ``vcs_push_id`` added to this table. I should probably add all these fields at once. These two fields should be populated by the etl layer and become (for now) optional fields in resultsets of the client.
Component: Treeherder → Treeherder: Data Ingestion
I will add the ``revision`` field in another bug 1199364
Attached file PR (deleted) —
Attachment #8672161 - Flags: review?(emorley)
Comment on attachment 8672161 [details] PR r=me, but left a comment
Attachment #8672161 - Flags: review?(emorley) → review+
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Depends on: 1304088
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: