• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

apache / parquet-cpp / 1039

Build:
DEFAULT BRANCH: master
Ran 30 Dec 2016 04:49PM UTC
Jobs 2
Files 1225
Run time 2min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

pending completion
1039

push

travis-ci

wesm
PARQUET-818: Refactoring to utilize common IO, buffer, memory management abstractions and implementations

This refactoring is a bit of a bloodbath, but I've attempted to preserve as much API backwards compatibility as possible.

Several points

* Arrow does not use exceptions, so will need to be very careful about making sure that no Status goes unchecked. I've tried to get most of them, but might have missed some
* parquet-cpp still exposes an abstract file read and write API as before, but this makes it easy to pass in an Arrow file handle (e.g. HDFS, OS files, memory maps, etc.)
* Custom memory allocators will need to subclass `arrow::MemoryPool` instead. If this becomes onerous for some reason, we can try to find alternatives, but basically it's the exact same class as `parquet::MemoryAllocator`

Does not require any upstream changes in Arrow.

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

Closes #210 from wesm/arrow-consolidation and squashes the following commits:

ef81084 [Wes McKinney] Configurable Arrow linkage. Slight .travis.yml cleaning
50b44f0 [Wes McKinney] Make some const refs
8438f86 [Wes McKinney] Revert ParquetFileReader::Open to use std::unique_ptr<RandomAccessFile>
671d981 [Wes McKinney] Actually tee output to console
ca8df13 [Wes McKinney] Do not hide test output from travis logs
f516115 [Wes McKinney] Add public link libs to dependencies to avoid race conditions with external projects
414c75f [Wes McKinney] README cleanups
be1acb5 [Wes McKinney] Move thirdparty ep's / setup to separate cmake module
46342ea [Wes McKinney] Remove unneeded ParquetAllocator interface, cleaning
b546f08 [Wes McKinney] Use MemoryAllocator alias within parquet core
8c1226d [Wes McKinney] Add Arrow to list of third party deps. Needs to be added to thirdparty
f9d8a2a [Wes McKinney] Check some unchecked Statuses
0d04820 [Wes McKinney] Fix benchmark builds. Do not fail in benchmarks if gtest.h is included due to <tr1/tuple> issue
ee312af [Wes McKinney] ... (continued)

556 of 556 new or added lines in 25 files covered. (100.0%)

10708 of 11502 relevant lines covered (93.1%)

141836.45 hits per line

New Missed Lines in Diff

Lines Coverage ∆ File
1
100.0
src/parquet/file/writer.cc
2
100.0
src/parquet/arrow/reader.cc
3
100.0
src/parquet/column/statistics.cc
21
100.0
src/parquet/util/memory.cc

Uncovered Existing Lines

Lines Coverage ∆ File
1
100.0
parquet-build/arrow_ep/src/arrow_ep-install/include/arrow/status.h
1
100.0
src/parquet/column/statistics.cc
1
100.0
parquet-build/arrow_ep/src/arrow_ep-install/include/arrow/util/bit-util.h
1
100.0
src/parquet/file/writer.cc
1
100.0
src/parquet/arrow/reader.cc
2
100.0
parquet-build/arrow_ep/src/arrow_ep-install/include/arrow/type.h
3
100.0
parquet-build/arrow_ep/src/arrow_ep-install/include/arrow/array.h
Jobs
ID Job ID Ran Files Coverage
1 1039.1 30 Dec 2016 04:49PM UTC 0
93.1
Travis Job 1039.1
2 1039.2 30 Dec 2016 04:51PM UTC 0
93.1
Travis Job 1039.2
Source Files on build 1039
Detailed source file information is not available for this build.
  • Back to Repo
  • Travis Build #1039
  • 2154e873 on github
  • Prev Build on master (#1032)
  • Next Build on master (#1041)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc