viirya opened a new issue, #4962:
URL: https://github.com/apache/arrow-datafusion/issues/4962

   **Is your feature request related to a problem or challenge? Please describe 
what you are trying to do.**
   
   In `Sum` aggregation expression, there are two state fields `sum` (int64) 
and `count` (uint64). For rust itself, it has no issue. But it makes that 
harder to integrate with external systems where only signed long is supported.
   
   Actually for `Sum` expression, we only care about if count is zero or not. 
We can definitely use a `bool` field there instead of a `count`.
   
   **Describe the solution you'd like**
   A clear and concise description of what you want to happen.
   
   **Describe alternatives you've considered**
   A clear and concise description of any alternative solutions or features 
you've considered.
   
   **Additional context**
   Add any other context or screenshots about the feature request here.
   


-- 
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...@arrow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to