ozankabak commented on PR #11845:
URL: https://github.com/apache/datafusion/pull/11845#issuecomment-2271007426

   I know this is still a draft, but can you remind us which issue/discussion 
this refactor is related to? Is it about removing logical stuff from 
aggregates? We can help more effectively with more context.
   
   Refactors are great when done judiciously but we should make sure we solve 
pressing pain points when we make major API changes or touch so many files at 
the same time.


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to