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

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/01/25 12:02:36 UTC

[GitHub] [arrow] paleolimbot commented on a change in pull request #12208: ARROW-14419 [R] Add filter + join test

paleolimbot commented on a change in pull request #12208:
URL: https://github.com/apache/arrow/pull/12208#discussion_r791642605



##########
File path: r/tests/testthat/test-dplyr-join.R
##########
@@ -173,3 +173,31 @@ test_that("mutate then join", {
     )
   )
 })
+
+test_that("filter then join", {

Review comment:
       Could you give this a slightly longer label? I'm not sure based on the title and content of the test what it is testing (without chasing the JIRAs back to the source).




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