-
Notifications
You must be signed in to change notification settings - Fork 320
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: aggregate reports based on the configured interval before sending to reporting #5326
feat: aggregate reports based on the configured interval before sending to reporting #5326
Conversation
…into feature/obs-415-support-aggregation-time-as-a-configuration-on-rudder-server
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #5326 +/- ##
==========================================
- Coverage 74.76% 74.74% -0.03%
==========================================
Files 433 433
Lines 60951 60979 +28
==========================================
+ Hits 45573 45581 +8
- Misses 12849 12863 +14
- Partials 2529 2535 +6 ☔ View full report in Codecov by Sentry. |
…s/rudder-server into feature/obs-415-support-aggregation-time-as-a-configuration-on-rudder-server
…s/rudder-server into feature/obs-415-support-aggregation-time-as-a-configuration-on-rudder-server
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.
LGTM
…s/rudder-server into feature/obs-415-support-aggregation-time-as-a-configuration-on-rudder-server
…s-a-configuration-on-rudder-server
…into feature/obs-415-support-aggregation-time-as-a-configuration-on-rudder-server
…s-a-configuration-on-rudder-server
…tion-on-rudder-server' of https://github.com/rudderlabs/rudder-server into feature/obs-415-support-aggregation-time-as-a-configuration-on-rudder-server
This reverts commit c3d0db9.
Description
Currently, the Rudder Server aggregates reports every minute and then flushes them to the Reporting Service. We want to make the aggregation interval configurable, allowing reports to be aggregated over a specified time period, such as 2 minutes instead of the default 1 minute.
Linear Ticket
https://linear.app/rudderstack/issue/OBS-415/support-aggregation-time-as-a-configuration-on-rudder-server-ability
Security