[GitHub] [arrow-datafusion] jgoday commented on pull request #505: Wrong aggregation arguments error.

[GitBox <gi...@apache.org>]

jgoday commented on pull request #505:
URL: https://github.com/apache/arrow-datafusion/pull/505#issuecomment-855016208


   @alamb Added this simple test (trying to create a physical plan with the example that you posted in the issue).
   
   ```rust
   async fn test_aggregation_with_bad_arguments() -> Result<()> {
       let mut ctx = ExecutionContext::new();
       register_aggregate_csv(&mut ctx)?;
       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);
       assert!(physical_plan.is_err());
       Ok(())
   }
   ```


-- 
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org