Omega359 commented on issue #17998:
URL: https://github.com/apache/datafusion/issues/17998#issuecomment-3590626748

   FYI @kosiew I've put a PR up on your fork 
(https://github.com/kosiew/datafusion/pull/31) that makes a bunch of changes 
and updates to your work. If you are unable to take a look at it in the next 
couple of weeks I'll create a PR here noting you as the originating author the 
work was based on.
   
   I've gone a long way with this issue in my local fork to refactor things 
even more and to add support for [pluggable datetime 
parsing](https://github.com/Omega359/arrow-datafusion/tree/jiff-proto) (Chrono 
by default, jiff as an option if feature enabled, with the possibility of 
others in the future (Java comes to mine for comet). Jiff is [faster overall 
for 
parsing](https://github.com/BurntSushi/jiff/blob/master/COMPARE.md#jiff-is-generally-faster-than-chrono)
 so it's likely something I'll use personally in the future. It's not quite as 
lenient as chrono though with some of the [strptime 
options](https://docs.rs/jiff/latest/jiff/fmt/strtime/index.html#conversion-specifications).


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to