-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
fix(eventstore): Use more correct start date for eventstore #67070
Conversation
Clickhouse behaves oddly when the normal epoch is used for dates: ClickHouse/ClickHouse#9335 (comment) This was working before by coincedence: an obsolete piece of code (query splitters) in Snuba was clamping the date to a non-epoch value. When the query splitters were removed, this started breaking. Instead of simply asking for "all time" use a date that aligns with how our data is actually stored. This is also not a new issue, see #30912
prev_filter.start = datetime.fromtimestamp(0) | ||
|
||
# We only store 90 days of data, add a few extra days just in case | ||
prev_filter.start = event.datetime - timedelta(days=100) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is quite a few days, I don't think I quite get why we need to go back so far
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Neither do I, but I didn't want to break any existing assumptions. The goal was to have this feature continue to work the way it was before.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #67070 +/- ##
=======================================
Coverage 84.27% 84.27%
=======================================
Files 5307 5307
Lines 237285 237301 +16
Branches 41054 41053 -1
=======================================
+ Hits 199973 199987 +14
- Misses 37094 37096 +2
Partials 218 218
|
Clickhouse behaves oddly when the normal epoch is used for dates: ClickHouse/ClickHouse#9335 (comment) This was working before by coincedence: an obsolete piece of code (query splitters) in Snuba was clamping the date to a non-epoch value. When the query splitters were removed, this started breaking. Instead of simply asking for "all time" use a date that aligns with how our data is actually stored. This is also not a new issue, see #30912
Clickhouse behaves oddly when the normal epoch is used for dates:
ClickHouse/ClickHouse#9335 (comment)
This was working before by coincedence: an obsolete piece of code (query
splitters) in Snuba was clamping the date to a non-epoch value. When the query
splitters were removed, this started breaking.
Instead of simply asking for "all time" use a date that aligns with how our
data is actually stored.
This is also not a new issue, see #30912