thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-24 08:32:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(prql): Robust panic handler

Browse Source
This commit is contained in:
Maximilian Roos 2024-02-29 17:43:06 -08:00
parent 0f6a1b451b
commit 1a5afa90a1
No known key found for this signature in database
GPG Key ID: 9058CC0B4BD47C30
2 changed files with 3 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
.gitignore vendored
View File

@ -165,7 +165,7 @@ tests/queries/0_stateless/*.expect.history
tests/integration/**/_gen tests/integration/**/_gen
# rust # rust
/rust/**/target /rust/**/target*
# It is autogenerated from *.in # It is autogenerated from *.in
/rust/**/.cargo/config.toml /rust/**/.cargo/config.toml
/rust/**/vendor /rust/**/vendor

10
rust/prql/src/lib.rs
View File

@ -57,14 +57,8 @@ pub unsafe extern "C" fn prql_to_sql(
out: *mut *mut u8, out: *mut *mut u8,
out_size: *mut u64, out_size: *mut u64,
) -> i64 { ) -> i64 {
let ret = panic::catch_unwind(|| { // NOTE: using cxxbridge we can return proper Result<> type.
return prql_to_sql_impl(query, size, out, out_size); panic::catch_unwind(|| prql_to_sql_impl(query, size, out, out_size)).unwrap_or(1)
});
return match ret {
// NOTE: using cxxbridge we can return proper Result<> type.
Err(_err) => 1,
Ok(res) => res,
}
} }
#[no_mangle] #[no_mangle]