eeroel commented on PR #37868:
URL: https://github.com/apache/arrow/pull/37868#issuecomment-1734854139

   > Can we use this part of interface: 
[4f8504f](https://github.com/apache/arrow/commit/4f8504f28877995ed179e35b077e6df6c14cac9c)
 ?
   
   Which part do you mean? In the current PR FileSource is constructed with 
FileInfos and FileInfos get constructed with size: 
https://github.com/apache/arrow/pull/37868/files#diff-82a2d9a8424dfb436344c0802dc36275cd0fb87e2e9eb6399745fba43799f7e5R63
   
   > By the way, `-1` is a bit misleading here...
   Yes, agreed it's not nice. I couldn't get the extension to compile when 
defaulting to size=None, got some errors about `Python object "cannot be passed 
as a varargs parameter"`. I'm not familiar with Cython so I don't really know 
what that's about...


-- 
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]

Reply via email to