1169367 - Combine event hit stats

Status: RESOLVED INVALID

(Webtools Graveyard :: Air Mozilla, defect, P2)

Description

[Peter Bengtsson [:peterbe]]

When we query Vid.ly for hits, we communicate by saying the Vid.ly shortcode (e.g. abc123). They give us a number (e.g. 12,345) of hits. 

We record this in EventHitStats which looks like this: https://github.com/mozilla/airmozilla/blob/d9f559f7b75aca79818dcad8c281f956e22f6d24/airmozilla/main/models.py#L698

That keeps is simple and we don't have to worry about doing increments.

However, a problem is that you might have 1 event with N shortcodes (where N is >= 1). 

For example, you decided to upload a different recording (e.g. one with some audio optimized for example). Now you might have had 1,000 views under the old Vid.ly video but 100 on the new Vid.ly video. It's still the same event. The true number should be 1,100. Not 100. 

If we're going to allow for editing of videos we might end up, often, with many vid.ly submissions for 1 event so we need to solve the statistics issue.

Comment 1

[Peter Bengtsson [:peterbe]]

Two possible solutions:

A) Change all places where we use EventHitStats and use aggregates instead. 

B) When we create a new EventHitStats record (where only the vidly shortcode is different), the carry the previous total_hits over somehow.

Comment 2

[Peter Bengtsson [:peterbe]]

Setting up a blocker on 1119986 because we have apparently let EventHitStats records be repeated that have the same event and shortcode.

Comment 3

[Andy Kochendorfer]

Platform deprecated.