fix: aggs cache ratio reporting (#8984)

Signed-off-by: loaki07 <rustyloaki@gmail.com>
This commit is contained in:
Loakesh Indiran
2025-11-06 22:07:59 +05:30
committed by GitHub
parent f7c9f6eb81
commit c7ee02a9f1
2 changed files with 4 additions and 8 deletions
+3 -6
View File
@@ -30,12 +30,9 @@ impl QueryCache for QueryCacheServerImpl {
request: Request<DeleteResultCacheRequest>,
) -> Result<Response<DeleteResultCacheResponse>, Status> {
let req: DeleteResultCacheRequest = request.into_inner();
let deleted = cacher::delete_cache(
&req.path, req.ts, None, // TODO: fix proper param
None, // TODO: fix proper param
)
.await
.is_ok();
let deleted = cacher::delete_cache(&req.path, req.ts, None, None)
.await
.is_ok();
Ok(Response::new(DeleteResultCacheResponse { deleted }))
}
+1 -2
View File
@@ -456,9 +456,8 @@ pub async fn run_datafusion(
.build()
)
);
let mut scan_stats = visit.scan_stats;
// Update scan stats to include aggregation cache ratio
scan_stats.aggs_cache_ratio = aggs_cache_ratio;
visit.scan_stats.aggs_cache_ratio = aggs_cache_ratio;
ret.map(|data| {
check_query_default_limit_exceeded(
data.iter().fold(0, |acc, batch| acc + batch.num_rows()),