Add new stream: ad_insights_hourly_advertiser #151
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of change
This PR adds a new Ad Insights Stream:
ads_insights_hourly_advertiser
.Notes
Excluded field
dda_results
is a field that appears in the Facebook Docs. However, I did not have test data for it and did not see anything in their docs about this field. So I have excluded it from being syncable in the tapBreakdown Behavior
Here's a blog post from Facebook announcing this new breakdown.
In that link, you'll see that you can request data for multiple days at a time.
For example, your request could contain
date_start = 2021-01-01
anddate_end = 2021-01-02
. While it's understandable that you get 48 data points out of this (one for each hour of the entire date range), that's not how this API works. It will aggregate all of the data for01:00 - 02:00
for every day in the date range.Because of this behavior, we have chosen to ask for a day's worth of data at a time, hoping that allows you to aggregate however you want in your warehouse.
Manual QA steps
full_view_reach
, so I took it out of the request.Risks
AdsInsight
StreamRollback steps