alamb commented on PR #7727:
URL: 
https://github.com/apache/arrow-datafusion/pull/7727#issuecomment-1747657892

   > I guess it is still possible to remove it, if we embed the data type info 
in to individual aggregate expression protobuf node. But this will involve a 
lot of changes just for removing the input_schema. 😄 So I stopped there and 
turned to add a few comment to explain why the parameter is there.
   
   Makes sense. I was thinking we could effectively recover the intermediate 
schema somehow (though I not quite sure how)
   
   


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