IMHO we could remove the call to `fopen` and the related check entirely. The 
resulting `file` is never used for reading/writing. It seems that replacing 
`fopen` with `open`/`_open` will give us a duplicate descriptor, and thus we 
will still have to close one of them: the new one of `fd`. If that's OK, before 
I replace `fopen` with `open`, could you please tell me why is it needed there 
in the first place?

[ Full content available at: https://github.com/apache/arrow/pull/2466 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to