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(analytics): Added load time metric for sdk, added percentile aggregate enum #4301
base: main
Are you sure you want to change the base?
Conversation
ba318e8
to
633b54c
Compare
83b0f8c
to
97158e2
Compare
@@ -7,7 +7,8 @@ use super::metrics::SdkEventMetricRow; | |||
pub struct SdkEventMetricsAccumulator { | |||
pub payment_attempts: CountAccumulator, | |||
pub payment_methods_call_count: CountAccumulator, | |||
pub average_payment_time: AverageAccumulator, | |||
pub average_payment_time: CountAccumulator, | |||
pub load_time: CountAccumulator, |
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.
is this correct with percentile? can you share a query?
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.
also why is average_payment_time changed to CountAccumulator?
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.
@ivor-juspay wanted to rename the metric to payment_time
since its now the median value
Type of Change
Description
Added load time metric for sdk, added percentile aggregate enum
Additional Changes
Motivation and Context
How did you test it?
Checklist
cargo +nightly fmt --all
cargo clippy