HyukjinKwon opened a new pull request, #48661:
URL: https://github.com/apache/arrow/pull/48661

   ### Rationale for this change
   
   03f8ae754e added a couple of TODOs for testing aggregation expressions
   
   
https://github.com/apache/arrow/blob/03f8ae754ede16f118ccdba0abb593b1461024aa/r/R/dplyr-filter.R#L49
   
   
https://github.com/apache/arrow/blob/03f8ae754ede16f118ccdba0abb593b1461024aa/r/R/dplyr-arrange.R#L54
   
   Should test negative cases too as mentioned in the TODOs.
   
   ### What changes are included in this PR?
   
   This PP adds a couple of tests for filter() and arrange() with aggregation 
expressions
   
   ### Are these changes tested?
   
   Unittests added.
   
   ### Are there any user-facing changes?
   
   No, test-only.


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