thisisnic commented on PR #33614: URL: https://github.com/apache/arrow/pull/33614#issuecomment-1383852321
> It looks like there are a few arguments to open_csv_dataset() that don't work, so maybe they should be omitted? This was a deliberate design decision whereby folks can easily just swap `read_csv_arrow()` and `open_csv_dataset()` calls, without changing parameters, and allowing for more informative error messages than `unused argument` if they found things that weren't compatible with both. I'm not too attached to it though - it doesn't feel like great practice either, and the arguments which are unsupported aren't the most widely-used. -- 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]
