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/03/16 14:03:20 UTC

[GitHub] [arrow] dragosmg edited a comment on pull request #12622: ARROW-15802 [R] bindings for lubridate::make_datetime() and lubridate::make_date()

dragosmg edited a comment on pull request #12622:
URL: https://github.com/apache/arrow/pull/12622#issuecomment-1069135304


   The are some outstanding issues, mostly having to do with the timezone argument:
   1. default values: 
   
     * for `ISOdate()` the default value is `"GMT"` which, in fact, is "UTC".
     * for `ISOdatetime()` the default value is `""` (current timezone) 
     * for `lubridate::make_datetime()` the default value is `"UTC"`
    
   2. the `strptime` binding does not support the `tz` argument - pointing to [ARROW-12820](https://issues.apache.org/jira/browse/ARROW-12820) - which is now solved. I have a feeling that issue doesn't actually mean we can now switch on `tz` support in `strptime`.
   
   For 1, the solution could be to set the default value for all `tz` args  to `"UTC"`.


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