ARROW-294: [C++] Do not use platform-dependent fopen/fclose functions for MemoryMappedFile #265

Closed
wants to merge 2 commits into
from

Projects

None yet

2 participants

@wesm
Member
wesm commented Jan 2, 2017

Also adds a test case for ARROW-340.

wesm added some commits Jan 2, 2017
@wesm wesm Base MemoryMappedFile implementation on common OSFile interface. Add …
…test case

for ARROW-340.

Change-Id: Iedbe5fe332dfb7007e7db8367b6adcbb73dbdecb
3928ab0
@wesm wesm Remove duplicated includes
Change-Id: I4e165909757790748b6d1aa8b2842963a4432e4f
42a83a4
@xhochy
xhochy approved these changes Jan 3, 2017 View changes

+1, LGTM

@asfgit asfgit pushed a commit that closed this pull request Jan 3, 2017
@wesm @xhochy wesm + xhochy ARROW-294: [C++] Do not use platform-dependent fopen/fclose functions…
… for MemoryMappedFile

Also adds a test case for ARROW-340.

Author: Wes McKinney <wes.mckinney@twosigma.com>

Closes #265 from wesm/ARROW-294 and squashes the following commits:

42a83a4 [Wes McKinney] Remove duplicated includes
3928ab0 [Wes McKinney] Base MemoryMappedFile implementation on common OSFile interface. Add test case for ARROW-340.
d9df556
@asfgit asfgit closed this in d9df556 Jan 3, 2017
@wesm wesm deleted the wesm:ARROW-294 branch Jan 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment