Skip to content
This repository was archived by the owner on May 10, 2024. It is now read-only.

Conversation

@xhochy
Copy link
Member

@xhochy xhochy commented May 1, 2016

Based on @emkornfield 's work in apache/arrow#29

PARQUET_TEST_ULIMIT_CORE=${PARQUET_TEST_ULIMIT_CORE:-0}
ulimit -c $PARQUET_TEST_ULIMIT_CORE
ARROW_TEST_ULIMIT_CORE=${ARROW_TEST_ULIMIT_CORE:-0}
ulimit -c $ARROW_TEST_ULIMIT_CORE
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Buglet from Arrow porting

@wesm
Copy link
Member

wesm commented May 1, 2016

Will review the benchmark code a bit later. Can you update the Travis CI config to only run the unit tests?

@xhochy
Copy link
Member Author

xhochy commented May 1, 2016

Benchmarks are now build but not run.

.travis.yml Outdated
before_script:
- source $TRAVIS_BUILD_DIR/ci/before_script_travis.sh
- cmake -DCMAKE_CXX_FLAGS="-Werror" -DPARQUET_TEST_MEMCHECK=ON -DPARQUET_GENERATE_COVERAGE=1 $TRAVIS_BUILD_DIR
- cmake -DCMAKE_CXX_FLAGS="-Werror" -DPARQUET_TEST_MEMCHECK=ON -DARROW_BUILD_BENCHMARKS=ON -DPARQUET_GENERATE_COVERAGE=1 $TRAVIS_BUILD_DIR

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

copy and paste bug?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes 😢

@wesm
Copy link
Member

wesm commented May 2, 2016

+1, thank you

@asfgit asfgit closed this in ff14d97 May 2, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants