rguerreiromsft commented on PR #3728: URL: https://github.com/apache/arrow-rs/pull/3728#issuecomment-1433633217
Thank you for both suggestions, I really appreciate them! I didn't remember about Borrow when coding it. About the RawReader, awesome, I'll start using it right away! However, I still need to use the `infer_json_schema_from_iterator` because I have to manually work with `serde_json::Value` before I attempt to generate the schema. Then I save this schema into a file to be used later, but I'm using tokio::fs to work with files, it'd be nicer to have the RawReader also support async work. It's not a huge issue, though, I can use std fs for this particular operation. On a separate note, I don't think exposing Values is a bad thing, sure a user doesn't need it, but when we're building a library on top of Arrow, it's nice to have access to lower level apis. -- 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]
