Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #1892 +/- ##
==========================================
- Coverage 98.40% 98.39% -0.02%
==========================================
Files 148 148
Lines 8886 8885 -1
==========================================
- Hits 8744 8742 -2
- Misses 142 143 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
st0012
left a comment
There was a problem hiding this comment.
Please also add a changelog entry, thx
| @pending_aggregates[session.aggregation_key] ||= init_aggregates(session.aggregation_key) | ||
| @pending_aggregates[session.aggregation_key][session.status] += 1 | ||
|
|
||
| aggregation_key = session.aggregation_key |
There was a problem hiding this comment.
Should the aggregation_key method memoizes its value then? Or should we rename it to generate_aggregation_key?
The fact that a *_key method generates different value every time it's called feels like a trap to me.
There was a problem hiding this comment.
wait no, I was just fixing this blindly. The key should not change because it always picks started, wtf.
There was a problem hiding this comment.
I'm not sure how else this error could happen then, I will memoize anyway and let's see if it fixes this user's problem.
403daeb to
dedd055
Compare
probably fixes #1891
(this is the only hypothesis I have currently for that error happening)