starred/openobserve
1
0
Fork 0
mirror of https://github.com/openobserve/openobserve.git synced 2026-11-06 08:09:24 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: do division first while calculating partitions (#6931)

Browse Source 
We see an overflow when we multiply the query duration to a large
storage size, to fix it we divide the storage size first with the data
retention time
This commit is contained in:
udev
2025-05-27 18:46:14 +05:30
committed by GitHub
parent 289d6234e5
commit ae7623f4bb
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/service/search/mod.rs
+2 -2
View File
@@ -697,8 +697,8 @@ pub async fn search_partition(
log::warn!("Data retention is zero, setting to 1 to prevent division by zero");
data_retention = 1;
}
let records = (stats.doc_num as i64 * query_duration) / data_retention;
let original_size = (stats.storage_size as i64 * query_duration) / data_retention;
let records = (stats.doc_num as i64 / data_retention) * query_duration;
let original_size = (stats.storage_size as i64 / data_retention) * query_duration;
log::info!(
"[trace_id {trace_id}] using approximation: stream: {}, records: {}, original_size: {}",
stream_name,