Skip to content

Commit 9287d55

Browse files
authoredMar 19, 2025
fix: datasets API edge case where stream has hot tier not enabled (#1252)
1 parent 6919a5c commit 9287d55

File tree

1 file changed

+9
-5
lines changed

1 file changed

+9
-5
lines changed
 

‎src/prism/logstream/mod.rs

+9-5
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ use chrono::Utc;
2424
use http::StatusCode;
2525
use serde::{Deserialize, Serialize};
2626
use serde_json::{json, Value};
27-
use tracing::{debug, warn};
27+
use tracing::{debug, error, warn};
2828

2929
use crate::{
3030
handlers::http::{
@@ -45,6 +45,7 @@ use crate::{
4545
arrow::record_batches_to_json,
4646
time::{TimeParseError, TimeRange},
4747
},
48+
validator::error::HotTierValidationError,
4849
LOCK_EXPECT,
4950
};
5051

@@ -273,10 +274,13 @@ impl PrismDatasetRequest {
273274
} = get_prism_logstream_info(stream).await?;
274275

275276
let hottier = match HotTierManager::global() {
276-
Some(hot_tier_manager) => {
277-
let stats = hot_tier_manager.get_hot_tier(stream).await?;
278-
Some(stats)
279-
}
277+
Some(manager) => match manager.get_hot_tier(stream).await {
278+
Ok(stats) => Some(stats),
279+
Err(HotTierError::HotTierValidationError(
280+
HotTierValidationError::NotFound(_),
281+
)) => None,
282+
Err(err) => return Err(err.into()),
283+
},
280284
_ => None,
281285
};
282286
let records = CountsRequest {

0 commit comments

Comments
 (0)