dragosmg commented on code in PR #13506:
URL: https://github.com/apache/arrow/pull/13506#discussion_r928541762
##########
r/R/dplyr-datetime-helpers.R:
##########
@@ -158,74 +158,40 @@ binding_as_date_numeric <- function(x, origin =
"1970-01-01") {
#'
#' @noRd
build_formats <- function(orders) {
- # only keep the letters and the underscore as separator -> allow the users to
- # pass strptime-like formats (with "%"). We process the data -> we need to
- # process the `orders` (even if supplied in the desired format)
- # Processing is needed (instead of passing
- # formats as-is) due to the processing of the character vector in
parse_date_time()
- orders <- gsub("[^A-Za-z]", "", orders)
Review Comment:
I'm not quite sure I follow why these 2 lines were removed.
--
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]
