From f942cd1bedac1b7af03898129a5403c5c4a7f28c Mon Sep 17 00:00:00 2001
From: pxLi <pxli@nyu.edu>
Date: Fri, 2 Apr 2021 22:52:12 +0800
Subject: [PATCH] Add USE_GDS as an option in build script (#7833)

Signed-off-by: Peixin Li <pxli@nyu.edu>

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: https://github.com/rapidsai/cudf/pull/7833
---
 java/ci/build-in-docker.sh | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/java/ci/build-in-docker.sh b/java/ci/build-in-docker.sh
index b2d0b066ce7..fc7e5762275 100755
--- a/java/ci/build-in-docker.sh
+++ b/java/ci/build-in-docker.sh
@@ -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
@@ -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"
 
@@ -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"