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

Add option to link Java bindings with Arrow dynamically [skip ci] #8307

Merged
merged 2 commits into from
May 24, 2021

Conversation

jlowe
Copy link
Member

@jlowe jlowe commented May 20, 2021

Closes #8300

Adds a Maven property, CUDF_JNI_ARROW_STATIC, that can be used to specify the JNI bindings should be linked against the Arrow library dynamically. This defaults to OFF since Arrow and its dependencies are not guaranteed to be available in the jar's environment, but this can be useful for cases where a developer has already built libcudf that dynamically links to Arrow and just wants to run Java tests within the same build environment without needing to rebuild libcudf with static Arrow.

For example, building and testing the Java bindings with dynamically-linked Arrow can be done via:

mvn clean test -DCUDF_JNI_ARROW_STATIC=OFF

@jlowe jlowe added 3 - Ready for Review Ready for review by team CMake CMake build issue Java Affects Java cuDF API. 4 - Needs cuDF (Java) Reviewer improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels May 20, 2021
@jlowe jlowe self-assigned this May 20, 2021
@jlowe jlowe requested a review from a team as a code owner May 20, 2021 22:29
java/pom.xml Show resolved Hide resolved
@jlowe jlowe changed the title Add option to link JNI bindings with Arrow dynamically [skip ci] Add option to link Java bindings with Arrow dynamically [skip ci] May 24, 2021
@jlowe
Copy link
Member Author

jlowe commented May 24, 2021

rerun tests

@jlowe
Copy link
Member Author

jlowe commented May 24, 2021

rerun tests

@jlowe
Copy link
Member Author

jlowe commented May 24, 2021

@gpucibot merge

@rapids-bot rapids-bot bot merged commit 66d6328 into rapidsai:branch-21.08 May 24, 2021
@jlowe jlowe deleted the jni-arrow-link-opt branch September 10, 2021 15:46
@vyasr vyasr added 4 - Needs Review Waiting for reviewer to review or respond and removed 4 - Needs cuDF (Java) Reviewer labels Feb 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 - Ready for Review Ready for review by team 4 - Needs Review Waiting for reviewer to review or respond CMake CMake build issue improvement Improvement / enhancement to an existing function Java Affects Java cuDF API. non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants