-
Notifications
You must be signed in to change notification settings - Fork 313
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
feat: new event type - record #3879
Conversation
@@ -16,6 +16,7 @@ const ( | |||
delimiter = "<<>>" | |||
eventStreamSourceCategory = "eventStream" | |||
extractEvent = "extract" | |||
rETLEvent = "record" |
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.
can record format be used by server side sdks too or do we want to restrict only to rETL?
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.
I would say rETL. Don't see how would it be useful for server side sdks.
a087428
to
7388e2e
Compare
a0da929
to
29acfab
Compare
cbdacab
to
be2e1d6
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3879 +/- ##
==========================================
- Coverage 71.98% 71.88% -0.11%
==========================================
Files 372 373 +1
Lines 54831 54834 +3
==========================================
- Hits 39472 39416 -56
- Misses 13059 13103 +44
- Partials 2300 2315 +15
☔ View full report in Codecov by Sentry. |
Description
New event type "record". (To be used to sync rETL records.)
Linear Ticket
https://linear.app/rudderstack/issue/INT-650/handling-new-event-type-processing-in-pipeline
Security