-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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(reduce transform): Fix flush not occuring when events arrive in high rate #16146
fix(reduce transform): Fix flush not occuring when events arrive in high rate #16146
Conversation
✅ Deploy Preview for vector-project canceled.
|
✅ Deploy Preview for vrl-playground canceled.
|
5d61c7e
to
7089a4f
Compare
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 agree that we need a fix here, and I provided an alternative to explore. We'll also need some unit testing of this behavior since it's subtle.
7089a4f
to
ac438f3
Compare
6a90502
to
cd08bca
Compare
cd08bca
to
b409ded
Compare
Regression Test Results
Run ID: f571c359-ad76-4bb2-afa7-c6a4fa309f9e Explanation
A regression test is an integrated performance test for
The table below, if present, lists those experiments that have experienced a
statistically significant change in their Changes in
Fine details of change detection per experiment.
|
@tomers looks like the test might've failed in other environments. maybe we need to play with the times? |
Fixes flush not occuring when events arrive in high frequency (in rate faster then `expire_after` between events). Fixes vectordotdev#16145.
b409ded
to
4eb6ff4
Compare
Regression Detector ResultsRun ID: 4d6ae017-e9c8-4b51-96fb-9f97f8c68742 ExplanationA regression test is an integrated performance test for The table below, if present, lists those experiments that have experienced a statistically significant change in their Changes in ingress throughput with confidence ≥ 90.00% and |Δ mean %| ≥ 5.00%:
Fine details of change detection per experiment.
|
Regression Detector ResultsRun ID: 83dae41e-320a-4eb0-8f29-170ac7c64ac9 ExplanationA regression test is an integrated performance test for The table below, if present, lists those experiments that have experienced a statistically significant change in their No interesting changes in Fine details of change detection per experiment.
|
6914d1f
to
f5e04c6
Compare
f5e04c6
to
b20a79c
Compare
@davidhuie-dd can you please approve latest changes? thanks |
Regression Detector ResultsRun ID: acfea22e-f473-4745-b214-26c1f31e856b ExplanationA regression test is an integrated performance test for The table below, if present, lists those experiments that have experienced a statistically significant change in mean optimization goal performance between baseline and comparison SHAs with 90.00% confidence OR have been detected as newly erratic. Negative values mean that baseline is faster, positive comparison. Results that do not exhibit more than a ±5.00% change in their mean optimization goal are discarded. An experiment is erratic if its coefficient of variation is greater than 0.1. The abbreviated table will be omitted if no interesting change is observed. Changes in experiment optimization goals with confidence ≥ 90.00% and |Δ mean %| ≥ 5.00%:
Fine details of change detection per experiment.
|
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 makes sense to me. Thanks!
Fixes flush not occuring when events arrive in high frequency (in rate faster then
expire_after
between events).Fixes #16145.