Jefffrey commented on code in PR #3209:
URL: https://github.com/apache/arrow-rs/pull/3209#discussion_r1033981345
##########
arrow-csv/src/reader.rs:
##########
@@ -90,10 +92,12 @@ fn infer_field_schema(string: &str, datetime_re:
Option<Regex>) -> DataType {
DataType::Float64
} else if INTEGER_RE.is_match(string) {
DataType::Int64
- } else if datetime_re.is_match(string) {
- DataType::Date64
- } else if DATE_RE.is_match(string) {
+ } else if DATE32_RE.is_match(string) {
Review Comment:
Ah didn't know about RegexSet, I can give it a shot in this PR a bit later
Edit: scratch that, might as well tackle as a separate PR instead
--
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]
