zeapo opened a new pull request #7309:
URL: https://github.com/apache/arrow/pull/7309


   A proposal of an implementation to read `json.gz` files. The idea is to 
along `.json` files, support gziped ones. 
   
   The implementation of `json::ReadBuilder` heavily rely on `File`. My first 
approach would have been to make another version of the builder that would rely 
on a `BufReader<Box<dyn Read>>` and have the original one delegate to this the 
final build.
   
   I went with another approach, in this case I've added an option to specify 
if the file is gziped or not. In this case, the original implementation works 
as defined, however requires a switch of type as per 
https://github.com/apache/arrow/compare/master...zeapo:feature/json-gzip?expand=1#diff-3fa6fc3c0ee201f5a3a1a5d25d0062ffR966-R967
 
   
   I've got no religion on this. Suggestions are welcome, to be honest, I would 
love that the `infer_json_schema` be public , which would make all of this code 
unnecessary as it would delegate the builder process to the users who care 
about these features.
   
   3 options :)


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


Reply via email to