Skip to content
This repository has been archived by the owner on Mar 24, 2021. It is now read-only.

Commit

Permalink
Filter by id - there is > 1 data point per slug
Browse files Browse the repository at this point in the history
Previous filtering by dasboard slug means it will not post if there is
ANY newer data point for that dashboard. This made sense when we posted
all data from a time regardless of nulls and so the latest timestamp of
one data point was representative of the latest data point for all the
others. This is no longer true as we now post the latest non null for a
dashboard slug.
  • Loading branch information
jcbashdown committed Jan 30, 2015
1 parent 0058974 commit 67aec76
Showing 1 changed file with 5 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -115,9 +115,12 @@ def _get_data_points_for_each_tx_metric(data, transform, data_set_config):
'data_type': transform['output']['data-type']},
transform,
datum,
# filter by record id as this is a hash of dashboard_slug
# and data_point_name and is therefore the important
# identifier of newer data
additional_read_params={
'filter_by': 'dashboard_slug:{}'.format(
datum['dashboard_slug'])}):
'filter_by': '_id:{}'.format(
datum['_id'])}):
yield datum


Expand Down

0 comments on commit 67aec76

Please sign in to comment.