anthonylouisbsb commented on pull request #9877: URL: https://github.com/apache/arrow/pull/9877#issuecomment-814159947
@projjal I found a limitation in the [arrow method](https://github.com/apache/arrow/blob/c2adaad3855a470c33b24189641008726ee63eed/cpp/src/arrow/util/value_parsing.h#L663) to convert string to dates and times using a defined format. That method calls the strptime method, which does not have an option to format sub-second granularities like milliseconds and microseconds, so if a user defines a format like YYYY-MM-DD HH:MI:SS.FFF, an error will be raised when the program tries to parse the date. [This example in my Pull Request](https://github.com/apache/arrow/blob/c2adaad3855a470c33b24189641008726ee63eed/cpp/src/gandiva/to_timestamp_holder_test.cc#L78) shows the error that occurs when I try to parse a date using milliseconds in the format string. Do I need to correct that problem in this Pull Request? Or it is better to create a new task to deal with the limitation and send the Pull Request with the possibility to format date, time and timestamps until the second time granularity now? -- 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: [email protected]
