liukun4515 commented on a change in pull request #1387:
URL: https://github.com/apache/arrow-datafusion/pull/1387#discussion_r763720460
##########
File path: datafusion/tests/sql.rs
##########
@@ -5612,10 +5612,9 @@ async fn test_aggregation_with_bad_arguments() ->
Result<()> {
let mut ctx = ExecutionContext::new();
register_aggregate_csv(&mut ctx).await?;
let sql = "SELECT COUNT(DISTINCT) FROM aggregate_test_100";
- let logical_plan = ctx.create_logical_plan(sql)?;
- let physical_plan = ctx.create_physical_plan(&logical_plan).await;
- let err = physical_plan.unwrap_err();
- assert_eq!(err.to_string(), "Error during planning: Invalid or wrong
number of arguments passed to aggregate: 'COUNT(DISTINCT )'");
+ let logical_plan = ctx.create_logical_plan(sql);
+ let err = logical_plan.unwrap_err();
+ assert_eq!(err.to_string(), DataFusionError::Plan("The function Count
expect argument number is 1, but the input argument number is
0".to_string()).to_string());
Review comment:
It will be changed to `Error during planning: The function Count expects
1 arguments, but 0 were provided`
After this comments
https://github.com/apache/arrow-datafusion/pull/1387/files#r760575015
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
