felipecrv commented on PR #15083:
URL: https://github.com/apache/arrow/pull/15083#issuecomment-1398443771

   > Some of these failing CI checks are a segmentation fault in the group_by 
method so I think they are relevant: 
https://github.com/apache/arrow/actions/runs/3944389169/jobs/6750236015
   
   @westonpace these are due to the removal of the checks against `i` that were 
suggested above. I reverted the commit, but that uncovered the fact that some 
extra input validation is needed. I'm investigating and will push a fix soon.
   
   > This PR also (automatically) exposes a "count_all" kernel in 
pyarrow.compute, but I am not sure how useful this is:
   
   @jorisvandenbossche I changed the python code to never expose nullary scalar 
aggregate functions.


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

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

Reply via email to