From 0ae2ec92303d4f78de450e466790141622b07307 Mon Sep 17 00:00:00 2001 From: paradox0503 Date: Thu, 7 Nov 2024 18:31:32 +0800 Subject: [PATCH] fix:limit error format --- datafusion/core/tests/sqllogictests/test_files/select.slt | 2 +- datafusion/sql/src/query.rs | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/datafusion/core/tests/sqllogictests/test_files/select.slt b/datafusion/core/tests/sqllogictests/test_files/select.slt index cc8828ef879c..c6d5867d39b7 100644 --- a/datafusion/core/tests/sqllogictests/test_files/select.slt +++ b/datafusion/core/tests/sqllogictests/test_files/select.slt @@ -231,7 +231,7 @@ select * from (select 1 a union all select 2) b order by a limit 1; 1 # select limit clause invalid -statement error DataFusion error: Error during planning: LIMIT must not be negative +statement error DataFusion error: Error during planning: LIMIT format is invalid select * from (select 1 a union all select 2) b order by a limit -1; # select limit clause diff --git a/datafusion/sql/src/query.rs b/datafusion/sql/src/query.rs index 2d7771d8c753..218859965490 100644 --- a/datafusion/sql/src/query.rs +++ b/datafusion/sql/src/query.rs @@ -143,7 +143,7 @@ impl<'a, S: ContextProvider> SqlToRel<'a, S> { Ok(n as usize) } _ => Err(DataFusionError::Plan( - "LIMIT must not be negative".to_string(), + "LIMIT format is invalid".to_string(), )), }?; Some(n)