Set a lower threshold for clippy to flag large error variants (#17109)

This commit is contained in:
Jeffrey Vo
2025-08-11 21:59:47 +10:00
committed by GitHub
parent c7d5904f01
commit c4f0375686
2 changed files with 12 additions and 2 deletions
+11 -1
View File
@@ -9,4 +9,14 @@ disallowed-types = [
# Lowering the threshold to help prevent stack overflows (default is 16384)
# See: https://rust-lang.github.io/rust-clippy/master/index.html#/large_futures
future-size-threshold = 10000
future-size-threshold = 10000
# Be more aware of large error variants which can impact the "happy path" due
# to large stack footprint when considering async state machines (default is 128).
#
# Value of 70 picked arbitrarily as something less than 100.
#
# See:
# - https://github.com/apache/datafusion/issues/16652
# - https://rust-lang.github.io/rust-clippy/master/index.html#result_large_err
large-error-threshold = 70
@@ -28,7 +28,7 @@ pub type Result<T, E = DFSqlLogicTestError> = std::result::Result<T, E>;
pub enum DFSqlLogicTestError {
/// Error from sqllogictest-rs
#[error("SqlLogicTest error(from sqllogictest-rs crate): {0}")]
SqlLogicTest(#[from] TestError),
SqlLogicTest(#[from] Box<TestError>),
/// Error from datafusion
#[error("DataFusion error: {}", .0.strip_backtrace())]
DataFusion(#[from] DataFusionError),