rok commented on pull request #12154: URL: https://github.com/apache/arrow/pull/12154#issuecomment-1013080010
This looks great @djnavarro thanks for doing it! Regarding your TODOs: > support week_start (essential) Perhaps you can put in an R shim to get lubridate matching logic and open a Jira to fix this in c++ later on. > support timezones correctly (essential) I found dates around 1900s quite problematic when testing against Pandas as they don't apply some historic offset changes. I'd suggest aiming at more modern times first. > support change_on_boundary argument to ceiling_date This would probably be a good c++ Jira candidate too. Provide some examples if you have them handy :). -- 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-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
