Uploaded image for project: 'Apache Arrow'
  1. Apache Arrow
  2. ARROW-13543

[R] Handle summarize() with 0 arguments or no aggregate functions

    XMLWordPrintableJSON

Details

    Description

      Following ARROW-13344, we should handle the case in which the user calls summarise() but does not call any aggregate functions in it.

      Currently this is unhandled and results in errors (which are then are converted to warnings) like:

      Warning: Error in x[c("fun", "options")] : object of type 'environment' is not subsettable ; pulling data into R

      or when the expression in summarise() has only scalar literals:

      Warning: Error : $ operator is invalid for atomic vectors
; pulling data into R

      or when there are zero arguments passed to summarise() and there is no group_by() preceding it:

      Warning: Error : Invalid input type, expected 'character' actual 'NULL'
; pulling data into R

      We should aim for consistency with dplyr::summarise() in these cases, but if that's not practical then we should detect this case and throw a more informative error.

      Attachments

        Issue Links

          blocks

          Improvement - An improvement or enhancement to an existing feature or task. ARROW-13618 [R] Use Arrow engine for summarize() by default

          • Critical - Crashes, loss of data, severe memory leak.
          • Resolved
          is depended upon by

          Wish - General wishlist item. ARROW-10415 [R] Support for dplyr::distinct()

          • Critical - Crashes, loss of data, severe memory leak.
          • Resolved
          relates to

          New Feature - A new feature of the product, which has yet to be developed. ARROW-13344 [R] Initial bindings for ExecPlan/ExecNode

          • Major - Major loss of function.
          • Resolved
          links to

          Web Link GitHub Pull Request #11078

          Activity

            People

              npr Neal Richardson
              icook Ian Cook
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 20m
                  20m