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

Testing for late-arriving facts #54

Open
jtcohen6 opened this issue Apr 1, 2019 · 1 comment
Open

Testing for late-arriving facts #54

jtcohen6 opened this issue Apr 1, 2019 · 1 comment

Comments

@jtcohen6
Copy link
Contributor

jtcohen6 commented Apr 1, 2019

Include in package integration tests event_update:

  • new event, same session_id as earlier event
  • new event, same page_view_id as earlier event (?)
  • same event_id as earlier event (?)

Our approach to these edge cases has been in flux over the past few releases. I think we have a good grasp of what desired behavior is as of 0.7.1. Let's include these cases in our integration tests to ensure the modeling operates as expected.

@jtcohen6 jtcohen6 added this to the 0.10.0 milestone Mar 27, 2020
@jtcohen6
Copy link
Contributor Author

This relates somewhat to #80.

I think it would be compelling to include a custom data test that returns, with warn/error thresholds (configurable?), the % of events that are "late-arriving" and % of sessions that are thereby "split." This would vary depending on the user's configured value of snowplow:page_view_lookback_days.

@jtcohen6 jtcohen6 removed this from the 0.11 milestone Jun 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant