alamb commented on issue #6139:
URL: 
https://github.com/apache/arrow-datafusion/issues/6139#issuecomment-1538573402

   > Thanks, I was expecting DISTINCT in plan, not a aggregation. But DISTINCT 
may be can considered as partial case of aggregation
   
   I agree that Distinct would be clearer in the LogicalPlan (though it gets 
rewritten to Aggregate pretty early on as I recall). Maybe it would be clearer 
to remove `LogicalPlan::Distinct` entirely and only use 
`LogicalPlan::Aggregate` 🤔 


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

Reply via email to