You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2020/04/23 07:39:15 UTC

[GitHub] [arrow] rdettai commented on issue #6949: ARROW-7681: [Rust] Explicitly seeking a BufReader will discard the internal buffer (2)

rdettai commented on issue #6949:
URL: https://github.com/apache/arrow/pull/6949#issuecomment-618235600


   I'll work on your comments today.
   
   What about the problem we are trying to fix here? Do you agree with the benefits of this fix ?
   
   Also, I'm not sure why a `Mutex` was used here. To we agree that `Refcell` is better as the whole reader is not thread safe, right ?


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org