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 2021/01/14 08:10:04 UTC

[GitHub] [arrow] jorisvandenbossche commented on pull request #9203: ARROW-11247: [C++] Infer date32 columns in CSV

jorisvandenbossche commented on pull request #9203:
URL: https://github.com/apache/arrow/pull/9203#issuecomment-760007070


   I added a commit that updates the tests for the new behaviour, *in case* we decide we are OK with that.
   
   Generally I think we should do the best inference from Arrow's point of view, and which is thus date type for a date string. 
   
   The only reason I am thinking to not do it is that, for people converting the data to pandas afterwards, dates are not that well supported (at this point) in pandas. Now, there is a `to_pandas(..., date_as_object=False)` keyword a user can specify to still get a datetime64 dtype in pandas instead of datetime.date objects.


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org