[ 
https://issues.apache.org/jira/browse/ARROW-12995?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17360238#comment-17360238
 ] 

Nate Clark commented on ARROW-12995:
------------------------------------

One of the validations I tried to add was that ReadOptions::block_size needs to 
be at least 1KB which seemed like a reasonable minimum but there are some csv 
tests which set block size to really small. I can update those tests to work 
with a larger block size or I can change the minimum block size to something 
like 1, any preferences?

> [C++] CSV reader should validate options
> ----------------------------------------
>
>                 Key: ARROW-12995
>                 URL: https://issues.apache.org/jira/browse/ARROW-12995
>             Project: Apache Arrow
>          Issue Type: Bug
>          Components: C++
>            Reporter: Nate Clark
>            Assignee: Nate Clark
>            Priority: Major
>
> Currently the only validation of csv options is done by asserts. These 
> options should always be validated by the reader on each reader creation.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to