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

[R] Helper function for casting from float to duration via int64()

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 9.0.0
    • R

    Description

      This logic is used in several places in R/dplyr-funcs-datetime.R and it would be great if we could separate it into a standalone helper function. The main purpose would be to indicate the float -> int64 -> duration casting chain is being used.

       

      Originally raised by Jonathan Keane during a review for one of the lubridate bindings.

      https://github.com/apache/arrow/pull/12757/#discussion_r852389088

      I've seen this line in a few PRs recently — this is to get around the fact that we do not have float durations, yeah? I wonder if we should put that into a helper function. Not so much to save lines of code, but to encapsulate that we have this one workaround that we're using in all of these places.

      Then if that workaround is not necessary (or if we find a better one) we can change it in one place. We don't need to do that here, but it would be good to make a Jira for that if we do want to.

      Attachments

        Issue Links

          Activity

            People

              dragosmg Dragoș Moldovan-Grünfeld
              dragosmg Dragoș Moldovan-Grünfeld
              Votes:
              0 Vote for this issue
              Watchers:
              4 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 - 1h 40m
                  1h 40m

                  Slack

                    Issue deployment