qstommyshu commented on code in PR #15548:
URL: https://github.com/apache/datafusion/pull/15548#discussion_r2027111797


##########
datafusion/sql/tests/sql_integration.rs:
##########
@@ -1766,41 +1767,56 @@ fn 
select_simple_aggregate_with_groupby_non_column_expression_and_its_column_sel
 
 #[test]
 fn select_simple_aggregate_nested_in_binary_expr_with_groupby() {
-    quick_test(
-        "SELECT state, MIN(age) < 10 FROM person GROUP BY state",
-        "Projection: person.state, min(person.age) < Int64(10)\
-             \n  Aggregate: groupBy=[[person.state]], aggr=[[min(person.age)]]\
-             \n    TableScan: person",
+    let plan =
+        generate_logical_plan("SELECT state, MIN(age) + 1 FROM person GROUP BY 
state");

Review Comment:
   Resolved.
   
   Thanks for the catch! I really need to be careful when using AI assist tool, 
I only used it for a few tests cases, but it already did quite some unexpected 
things... I guess AI is still not mature enough for this.



-- 
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: github-unsubscr...@datafusion.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to