Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release snowplow media player/0.9.1 #83

Merged
merged 3 commits into from
Oct 23, 2024

Conversation

ilias1111
Copy link
Contributor

Description & motivation

Summary

This patch release adds app_id as a new field in some of the derived tables and includes a bug fix related to user identification handling in ad_views.

Features

  • Introduced app_id field across some of the derived tables to improve application-level analytics

Fix

  • Optimized user_id aggregation, changed from GROUP BY to MAX(user_id) in queries in ad_view, because it could cause duplicates in an edge case.

Upgrading

To upgrade, update the snowplow-media-player version in your packages.yml file. Please note:

  • The new app_id field will be populated for new data points automatically
  • Although this is a non-breaking change, historical data will require a full refresh to include app_id values, if needed

Checklist

  • I have verified that these changes work locally
  • I have updated the README.md (if applicable)
  • I have added tests & descriptions to my models (and macros if applicable)
  • I have raised a documentation PR

OscarSnowPlow and others added 3 commits October 21, 2024 11:58
Added app_id column to models
In essence we are going to treat user_id the same way we treat domain_sessionid_array, as we may have more than one value and this will cause unwanted extra rows.
@ilias1111 ilias1111 marked this pull request as ready for review October 23, 2024 13:11
@ilias1111 ilias1111 requested a review from a team as a code owner October 23, 2024 13:11
@ilias1111 ilias1111 merged commit 7801e68 into main Oct 23, 2024
1 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants