[
https://issues.apache.org/jira/browse/ARROW-1641?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16192116#comment-16192116
]
ASF GitHub Bot commented on ARROW-1641:
---------------------------------------
Github user wesm commented on the issue:
https://github.com/apache/arrow/pull/1165
Hm, seems I have some more work to do here. I'm not sure about the best
strategy for threadsafety by default in the `RandomAccessFile::ReadAt`
functions. Before there was a default virtual implementation, but the mutex was
visible in the header. We could add a PIMPL to `RandomAccessFile` but that
might be overkill.
The default option is that I add basic implementations to the classes that
are breaking the build currently
> [C++] Do not include <mutex> in public headers
> ----------------------------------------------
>
> Key: ARROW-1641
> URL: https://issues.apache.org/jira/browse/ARROW-1641
> Project: Apache Arrow
> Issue Type: Improvement
> Components: C++
> Reporter: Wes McKinney
> Assignee: Wes McKinney
> Labels: pull-request-available
> Fix For: 0.8.0
>
>
> This is a part of ARROW-1134
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
