You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by "ianmcook (via GitHub)" <gi...@apache.org> on 2023/02/03 00:14:33 UTC

[GitHub] [arrow] ianmcook commented on pull request #33917: GH-33892: [R] Map dplyr::n() to count_all kernel

ianmcook commented on PR #33917:
URL: https://github.com/apache/arrow/pull/33917#issuecomment-1414539013

   @nealrichardson thanks for your review! I think I addressed or responded to all the issues you raised. I also changed some of the helper functions so that they return types instead of Expressions. They are only ever used to assemble the schema, so it is clearer this way.


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

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