-
Notifications
You must be signed in to change notification settings - Fork 301
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(warehouse): grouping uploads for processing pickup #3039
Conversation
…re.processor-pickup-first_event_at
…re.processor-pickup-first_event_at
69921a5
to
2861ce2
Compare
@@ -140,6 +143,364 @@ func TestUploads_Get(t *testing.T) { | |||
}) | |||
} | |||
|
|||
func TestUploads_GetToProcess(t *testing.T) { |
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.
[minor] Some of the tests presented here could have been combined with TestUploads_Processing
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.
Missed that test. Because of time constraints, Will merge both of the tests in separate PR.
Codecov ReportBase: 53.00% // Head: 52.97% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #3039 +/- ##
==========================================
- Coverage 53.00% 52.97% -0.03%
==========================================
Files 334 334
Lines 51873 51874 +1
==========================================
- Hits 27493 27479 -14
- Misses 22782 22793 +11
- Partials 1598 1602 +4
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Description
Use first_event_at for grouping while pickup for warehouse processing jobs
Notion Ticket
https://www.notion.so/rudderstacks/Use-first_event_at-while-pickup-from-processing-31c841fcdecc4ea8b6139ee0cb4d8daa?pvs=4
Security