Skip to content

Commit

Permalink
Add USE_GDS as an option in build script (#7833)
Browse files Browse the repository at this point in the history
Signed-off-by: Peixin Li <[email protected]>

add ENABLE_GDS as an argument of the build script. We keep it OFF as default since build w/ GDS will require specific GDS lib installed correctly on the build machine, and we can turn that on in our pipelines.

Authors:
  - pxLi (https://github.com/pxLi)

Approvers:
  - Rong Ou (https://github.com/rongou)
  - Jason Lowe (https://github.com/jlowe)

URL: #7833
  • Loading branch information
pxLi authored Apr 2, 2021
1 parent 6c536b5 commit f942cd1
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion java/ci/build-in-docker.sh
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ BUILD_CPP_TESTS=${BUILD_CPP_TESTS:-OFF}
ENABLE_PTDS=${ENABLE_PTDS:-ON}
RMM_LOGGING_LEVEL=${RMM_LOGGING_LEVEL:-OFF}
ENABLE_NVTX=${ENABLE_NVTX:-ON}
ENABLE_GDS=${ENABLE_GDS:-OFF}
OUT=${OUT:-out}

SIGN_FILE=$1
Expand All @@ -37,6 +38,7 @@ echo "SIGN_FILE: $SIGN_FILE,\
BUILD_CPP_TESTS: $BUILD_CPP_TESTS,\
ENABLED_PTDS: $ENABLE_PTDS,\
ENABLE_NVTX: $ENABLE_NVTX,\
ENABLE_GDS: $ENABLE_GDS,\
RMM_LOGGING_LEVEL: $RMM_LOGGING_LEVEL,\
OUT_PATH: $OUT_PATH"

Expand All @@ -58,7 +60,7 @@ make -j$PARALLEL_LEVEL
make install DESTDIR=$INSTALL_PREFIX

###### Build cudf jar ######
BUILD_ARG="-Dmaven.repo.local=$WORKSPACE/.m2 -DskipTests=$SKIP_JAVA_TESTS -DPER_THREAD_DEFAULT_STREAM=$ENABLE_PTDS -DRMM_LOGGING_LEVEL=$RMM_LOGGING_LEVEL"
BUILD_ARG="-Dmaven.repo.local=$WORKSPACE/.m2 -DskipTests=$SKIP_JAVA_TESTS -DPER_THREAD_DEFAULT_STREAM=$ENABLE_PTDS -DRMM_LOGGING_LEVEL=$RMM_LOGGING_LEVEL -DUSE_GDS=$ENABLE_GDS"
if [ "$SIGN_FILE" == true ]; then
# Build javadoc and sources only when SIGN_FILE is true
BUILD_ARG="$BUILD_ARG -Prelease"
Expand Down

0 comments on commit f942cd1

Please sign in to comment.