marsupialtail commented on PR #13640: URL: https://github.com/apache/arrow/pull/13640#issuecomment-1231146444
The Tell() for DirectFileOutputStream is a bit annoying. You could have its behavior to be return the bytes written to the stream, including the cached length, or the bytes actually written to the file. Other OutputStream implementations don't have this distinction. Taking either route will fail some tests in our current test suite. I chose the latter approach since it fails fewer tests (1) vs the former (13). I just removed the ASSERT that fails the latter approach. -- 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]
