dragosmg commented on pull request #12429: URL: https://github.com/apache/arrow/pull/12429#issuecomment-1041378890
Not sure if we should (and if _yes_, at what level) support integer / numeric arguments to `month()`. If the answer is _yes_, I feel that should not happen only inside / for semester. `lubridate::month()` has dual purpose - it can be used both to get and set months. arrow currently supports only extraction and, therefore, the input type needs to be `temporal`. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
