You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2018/12/15 16:59:42 UTC

[GitHub] MaxGekk commented on issue #23202: [SPARK-26248][SQL] Infer date type from CSV

MaxGekk commented on issue #23202: [SPARK-26248][SQL] Infer date type from CSV
URL: https://github.com/apache/spark/pull/23202#issuecomment-447582380
 
 
   I have rebased this branch on the master and as a consequence of that `CsvInferSchema` uses new date/timestamp parser for type inference. Can we continue with this PR since it is used new `Date`/`TimeFormatter` introduced by https://github.com/apache/spark/pull/23150 and probably will be not affected by https://github.com/apache/spark/pull/23196. 
   
   Also I changed order of type inference here. For now `TimestampType` is inferred before `DateType`. /cc @cloud-fan 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org