shehabgamin commented on PR #17561: URL: https://github.com/apache/datafusion/pull/17561#issuecomment-3331330447
> > Tried taking an initial look but the PR is quite massive so a full review will take quite a while. I left some initial notes, but some more overall notes: > > @andygrove or @shehabgamin -- do you have any insight to how other rust libraries handle java formatting strings? Is this something that already exists in the comet or Lakesail codebase that we could migrate (or borrow tests for?) Sail doesn't have `format_string` implemented yet, but if helpful: Here are all the string functions that Sail has implemented: - https://docs.lakesail.com/sail/main/guide/functions/scalar.html#string And here is the actual code for all of the Sail functions: - https://github.com/lakehq/sail/tree/5d91c408a2bba2d2da691c3c25400a9708ea4050/crates/sail-plan/src/function - https://github.com/lakehq/sail/tree/5d91c408a2bba2d2da691c3c25400a9708ea4050/crates/sail-function/src Feel free to migrate/take whatever, all of the functions that Sail has implemented are going to be donated to DataFusion overtime anyway! -- 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]
