davlee1972 commented on issue #13111:
URL: https://github.com/apache/arrow/issues/13111#issuecomment-1644683478

   **Wouldn't it be cleaner to add a  locale parameter to strptime?**
   
   pyarrow.compute.strptime(strings, /, format, unit, error_is_null=False, *, 
options=None, memory_pool=None)
   
   **strftime does support a locale parameter to convert timestamps to string**
   
   pyarrow.compute.strftime(timestamps, /, format='%Y-%m-%dT%H:%M:%S', 
**locale='C'**, *, options=None, 
memory_pool=None)[ΒΆ](https://arrow.apache.org/docs/python/generated/pyarrow.compute.strftime.html#pyarrow.compute.strftime)
   
   **This would also need to be added to pyarrow.csv.ConvertOptions. The 
timestamp_parsers current is just a list of strptime format strings without any 
type of locale.**
   
   
https://arrow.apache.org/docs/python/generated/pyarrow.csv.ConvertOptions.html#pyarrow-csv-convertoptions
   
   _timestamp_parsers_
   A sequence of strptime()-compatible format strings, tried in order when 
attempting to infer or convert timestamp values (the special value ISO8601() 
can also be given). By default, a fast built-in ISO-8601 parser is used.


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

Reply via email to