Rich-T-kid commented on PR #21765:
URL: https://github.com/apache/datafusion/pull/21765#issuecomment-4290601724

   @alamb this PR should resolve the initial regression that was caused 
[(described here)](https://github.com/apache/datafusion/issues/8738) as well as 
provide performance boost. I would love to hear your thoughts this approach
   
   > I think we need to create a benchmark that does aggregation queries on 
dictionary encoded string columns (I know the existing end to end TPCH, and 
ClickBench benchmarks do not do this). I will file a ticket shortly about this 
   
[source](https://github.com/apache/datafusion/issues/7647#issuecomment-1880919163)
 
   
   seeing as how this PR is already large, I think this would be a nice follow 
up PR since as far as i know this work was never finished since #9017 was 
closed due to inactivity


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