Omega359 commented on code in PR #15242:
URL: https://github.com/apache/datafusion/pull/15242#discussion_r2010222655


##########
datafusion/sql/tests/sql_integration.rs:
##########
@@ -1901,8 +1901,9 @@ fn union_by_name_different_columns() {
         \n    Projection: NULL AS Int64(1), order_id\
         \n      Projection: orders.order_id\
         \n        TableScan: orders\
-        \n    Projection: orders.order_id, Int64(1)\
-        \n      TableScan: orders";
+        \n    Projection: Int64(1), order_id\

Review Comment:
   I'm trying to figure out where this Projection (`Projection: Int64(1), 
order_id`) is coming from and why it's ordered this way. I would expect it to 
be ordered as order_id, Int64 as that is the order on the rhs.
   
   As well, this test has an expected distinct clause  ... the query doesn't. I 
can't see a reason why that would be introduced.



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