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

ASF GitHub Bot commented on ARROW-2036:
---------------------------------------

jcrist commented on issue #1517: ARROW-2036: [Python] Support standard IOBase 
methods on NativeFile
URL: https://github.com/apache/arrow/pull/1517#issuecomment-360810013
 
 
   Sounds good, I suspect we're just talking past each other here. Apologies 
for sidetracking this PR.
   
   In regards to this PR, I'm a bit unsure about the `writeable -> writable` 
change. If you google "writeable or writable" you'll find other projects(e.g. 
nodejs) that have also gone through this decision - most settled on `writable` 
as it's a more common english spelling. I made this change originally to mirror 
the added `writable` method (part of the python file interface). However, a few 
dependencies of pyarrow (`libhdfs` and `numpy`) use "writeable" instead, as 
does the arrow c++ code. I could go either way on this change.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


> NativeFile should support standard IOBase methods
> -------------------------------------------------
>
>                 Key: ARROW-2036
>                 URL: https://issues.apache.org/jira/browse/ARROW-2036
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: Python
>            Reporter: Jim Crist
>            Priority: Minor
>              Labels: pull-request-available
>
> If `NativeFile` supported most/all of the standard IOBase methods 
> ([https://docs.python.org/3/library/io.html#io.IOBase),] then it'd be easier 
> to use arrow files with other python libraries. Would at least be nice to 
> support enough operations to use `io.TextIOWrapper`.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to