[GitHub] [arrow] westonpace opened a new issue, #35211: [C++] Simplify "FileFormat options" vs. "Fragment scan/write options"

["westonpace (via GitHub)" <gi...@apache.org>]

westonpace opened a new issue, #35211:
URL: https://github.com/apache/arrow/issues/35211

   ### Describe the enhancement requested
   
   Some file formats (maybe just the parquet format) have configuration options on the format directly.  For example, ParquetFileFormat::reader_options (and soon there will be encryption configuration).  However, we also have the base FileFormat::default_fragment_scan_options.  This makes it confusing for users who have to look through and understand several different places to configure options.
   
   We should pick one place and put all the configuration for reads there.  I think the easiest approach is to stick with `FileFormat::default_fragment_scan_options`.  Then the `FileFormat` subclasses should, themselves, be stateless (their parent class will have state but the subclasses will add no additional member variables).
   
   
   
   ### Component(s)
   
   C++


-- 
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: issues-unsubscribe@arrow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org