-
Notifications
You must be signed in to change notification settings - Fork 5.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make Bazel the default build system #3898
Conversation
Test FAILed. |
Test PASSed. |
Test FAILed. |
Test FAILed. |
Test FAILed. |
Test FAILed. |
Test FAILed. |
Test PASSed. |
Test FAILed. |
Test PASSed. |
Test FAILed. |
Test FAILed. |
Test PASSed. |
It look like the Java dynamic lib build should be also added to the bazel building process. |
@guoyuhong: Yes, can you contribute this in a separate PR? I'm not too familiar with Java build systems, but Bazel should support it well. |
@guoyuhong I'm not familiar with Bazel... I will have a try. |
Test FAILed. |
Test FAILed. |
Test FAILed. |
Test FAILed. |
Test PASSed. |
Test FAILed. |
Test FAILed. |
Test PASSed. |
Test FAILed. |
Test PASSed. |
Test FAILed. |
Test FAILed. |
Test FAILed. |
Test FAILed. |
Test FAILed. |
Test FAILed. |
Test FAILed. |
Jenkins retest this please |
Test FAILed. |
build.sh
Outdated
# The following line installs pyarrow from S3, these wheels have been | ||
# generated from https://github.com/ray-project/arrow-build from | ||
# the commit listed in the command. | ||
pip install --target=$ROOT_DIR/python/ray/pyarrow_files pyarrow==0.12.0-RAY \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will find wrong pip in some cases
build.sh
Outdated
# generated from https://github.com/ray-project/arrow-build from | ||
# the commit listed in the command. | ||
pip install --target=$ROOT_DIR/python/ray/pyarrow_files pyarrow==0.12.0-RAY \ | ||
--find-links https://s3-us-west-2.amazonaws.com/arrow-wheels/bf0f3a15e0d6583cfaedbca508627b06746fa41a/index.html |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In this current form this drops support for 3.4, so we should either build Arrow wheels for 3.4 or discuss that.
@pcmoritz can you file an issue describing the C++ issues that we run into with the Arrow version in this PR? |
build.sh
Outdated
# generated from https://github.com/ray-project/arrow-build from | ||
# the commit listed in the command. | ||
pip install --target=$ROOT_DIR/python/ray/pyarrow_files pyarrow==0.12.0-RAY \ | ||
--find-links https://s3-us-west-2.amazonaws.com/arrow-wheels/bf0f3a15e0d6583cfaedbca508627b06746fa41a/index.html |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Either in this PR or a subsequent one, it will be important to document how to update the Arrow version.
Test FAILed. |
Test FAILed. |
Test FAILed. |
4cc4961
to
c908280
Compare
c908280
to
cb7888a
Compare
Test FAILed. |
Test FAILed. |
Test FAILed. |
Progress on #2887. |
What do these changes do?
This switches the build system from CMake to Bazel for developers.
The wheels, valgrind tests and Jenkins are currently still run with CMake and will be switched in follow up PRs.
Related issue number