PARQUET-827: Account for arrow::MemoryPool API change and fix bug in reading Int96 timestamps #215

Closed
wants to merge 5 commits into
from

Projects

None yet

2 participants

@wesm
Member
wesm commented Jan 7, 2017 edited

Prior to ARROW-427, the "length()" field was not being properly checked in the implementation of PrimitiveArray::Equals.

@wesm wesm changed the title from PARQUET-825: Account for arrow::MemoryPool API change and fix bug in reading Int96 timestamps to PARQUET-827: Account for arrow::MemoryPool API change and fix bug in reading Int96 timestamps Jan 7, 2017
@xhochy
xhochy approved these changes Jan 7, 2017 View changes

+1, LGTM

Possibly we want to move this allocator to Arrow or add this functionality to the existing ones.

@wesm
Member
wesm commented Jan 7, 2017

agreed

@asfgit asfgit pushed a commit that closed this pull request Jan 7, 2017
@wesm wesm PARQUET-827: Account for arrow::MemoryPool API change and fix bug in …
…reading Int96 timestamps

Prior to ARROW-427, the "length()" field was not being properly checked in the implementation of `PrimitiveArray::Equals`.

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

Closes #215 from wesm/PARQUET-825 and squashes the following commits:

afa41f9 [Wes McKinney] Do not build arrow_jemalloc in arrow external project
ff9f22e [Wes McKinney] cpplint, clang-format
802b325 [Wes McKinney] Update Arrow version
5f155b6 [Wes McKinney] Fix bug exposed by accidental bug fix in ARROW-427
a3c75bb [Wes McKinney] Implement TrackingAllocator based on Arrow default allocator
fb9c116
@asfgit asfgit closed this in fb9c116 Jan 7, 2017
@wesm wesm deleted the wesm:PARQUET-825 branch Jan 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment