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

fix: return proper error for incorrect function panic (#7708) (#8021)

Browse Source
This commit is contained in:
Shivank Anchal
2025-08-19 19:45:02 +08:00
committed by GitHub
parent cd00d5b06c
commit 111a67b73d
1 changed files with 14 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/service/search/cluster/http.rs
+14 -5
View File
@@ -177,14 +177,18 @@ pub async fn search(
&sql.org_id,
&stream_names,
);
ret_val
.as_array()
.unwrap()
.ok_or(Error::Message("Expected array".to_string()))?
.iter()
.filter_map(|v| {
(!v.is_null()).then_some(flatten::flatten(v.clone()).unwrap())
(!v.is_null()).then_some(flatten::flatten(v.clone()).map_err(|e| {
log::error!("Failed to flatten value: {}", e);
Error::Message(format!("Failed to flatten value: {e}"))
}))
})
.collect()
.collect::<Result<Vec<_>, _>>()?
} else {
json_rows
.into_iter()
@@ -200,9 +204,14 @@ pub async fn search(
&sql.org_id,
&stream_names,
);
(!ret_val.is_null()).then_some(flatten::flatten(ret_val).unwrap())
(!ret_val.is_null()).then_some(flatten::flatten(ret_val).map_err(
|e| {
log::error!("Failed to flatten value: {}", e);
Error::Message(format!("Failed to flatten value: {e}"))
},
))
})
.collect()
.collect::<Result<Vec<_>, _>>()?
}
}
None => json_rows