Skip to content
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

Update cudf dependency to 0.16-SNAPSHOT #727

Merged
merged 3 commits into from
Sep 15, 2020

Conversation

jlowe
Copy link
Member

@jlowe jlowe commented Sep 10, 2020

This updates the cudf dependency to 0.16-SNAPSHOT.

Marking this as a draft for now, as it appears somehow the libcudf.so in the current cudf-0.16-SNAPSHOT says it requires librmm.so which fails the build because the jar does not contain librmm.so. Investigating.

@jlowe jlowe added the build Related to CI / CD or cleanly building label Sep 10, 2020
@jlowe jlowe self-assigned this Sep 10, 2020
@jlowe jlowe changed the base branch from branch-0.2 to branch-0.3 September 10, 2020 19:27
revans2
revans2 previously approved these changes Sep 10, 2020
@jlowe
Copy link
Member Author

jlowe commented Sep 11, 2020

The librmm.so issue with cudf-0.16-SNAPSHOT has been resolved, this should be ready to go.

@jlowe jlowe marked this pull request as ready for review September 11, 2020 15:47
@jlowe
Copy link
Member Author

jlowe commented Sep 11, 2020

build

tgravescs
tgravescs previously approved these changes Sep 11, 2020
@jlowe
Copy link
Member Author

jlowe commented Sep 14, 2020

Waiting on rapidsai/cudf#6215 to cleanup the rmm_log.txt files that are failing the CI build.

revans2
revans2 previously approved these changes Sep 14, 2020
@jlowe
Copy link
Member Author

jlowe commented Sep 15, 2020

It might take a while to get the rmm_log.txt issue fully resolved, so for now I excluded it from the rat checks. If this is acceptable as a short-term solution, I'll file a followup issue to have it removed when cudf is no longer emitting this file by default.

@jlowe
Copy link
Member Author

jlowe commented Sep 15, 2020

build

@jlowe jlowe merged commit e84b923 into NVIDIA:branch-0.3 Sep 15, 2020
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
* Update cudf dependency to 0.16-SNAPSHOT

Signed-off-by: Jason Lowe <jlowe@nvidia.com>

* Update docs to reflect new artifact versions

* Exclude rmm_log.txt
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
* Update cudf dependency to 0.16-SNAPSHOT

Signed-off-by: Jason Lowe <jlowe@nvidia.com>

* Update docs to reflect new artifact versions

* Exclude rmm_log.txt
tgravescs pushed a commit to tgravescs/spark-rapids that referenced this pull request Nov 30, 2023
[auto-merge] bot-auto-merge-branch-22.12 to branch-23.02 [skip ci] [bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Related to CI / CD or cleanly building
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants