rust/hyperswitch
1
0
Fork 0
mirror of https://github.com/juspay/hyperswitch.git synced 2025-11-03 05:17:02 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix(analytics): add dynamic limit by clause in failure reasons metric query (#6462)

Browse Source
This commit is contained in:
Sandeep Kumar
2024-11-04 19:49:23 +05:30
committed by GitHub
parent 19cf0f7437
commit 88253780d7
1 changed files with 11 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
crates/analytics/src/payments/metrics/sessionized_metrics/failure_reasons.rs
View File

@ -148,17 +148,20 @@ where
.attach_printable("Error adding order by clause") .attach_printable("Error adding order by clause")
.switch()?; .switch()?;
for dim in dimensions.iter() { let filtered_dimensions: Vec<&PaymentDimensions> = dimensions
if dim != &PaymentDimensions::ErrorReason { .iter()
outer_query_builder .filter(|&&dim| dim != PaymentDimensions::ErrorReason)
.add_order_by_clause(dim, Order::Ascending) .collect();
.attach_printable("Error adding order by clause")
.switch()?; for dim in &filtered_dimensions {
} outer_query_builder
.add_order_by_clause(*dim, Order::Ascending)
.attach_printable("Error adding order by clause")
.switch()?;
} }
outer_query_builder outer_query_builder
.set_limit_by(5, &[PaymentDimensions::Connector]) .set_limit_by(5, &filtered_dimensions)
.attach_printable("Error adding limit clause") .attach_printable("Error adding limit clause")
.switch()?; .switch()?;