You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2023/01/12 11:52:21 UTC

[GitHub] [arrow] thisisnic commented on issue #31355: [R] CsvConvertOptions include_columns and col_select should give better error message when used in open_dataset

thisisnic commented on issue #31355:
URL: https://github.com/apache/arrow/issues/31355#issuecomment-1380214475

   I'm closing this issue due to the fact that we recently merged #15270 which allows users to pass in read/parse/convert options as lists - when users do this, we can manually check the values in these lists to see if they're compatible with the datasets API or not.  We cannot do this with Csv*Options objects.  We should consider direct instantiation of Csv*Options etc objects "off-label" usage, and encourage passing in items as lists only.


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

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