niyue commented on pull request #11486: URL: https://github.com/apache/arrow/pull/11486#issuecomment-956937628
@westonpace @lidavidm Instead of using mockfs, I simplified the unit testing by using the `TrackedRandomAccessFile` suggested by David, now there is no change for CMake file, and there is no change to the `io::BufferReader` API. Could you please help to review? BTW, is there any documentation describing how I can run the `clang-format` like the CI job? I find my change sometimes breaks CI lint job, but I've no idea what the recommended approach is for running clang format locally for this project. -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
