Review Comments

NVIDIA · May 6, 2022 · 51596da · 51596da
1 parent d494746
commit 51596da
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 44 deletions.
diff --git a/build-libcudf.xml b/build-libcudf.xml
@@ -42,7 +42,7 @@
       <arg value="-DCMAKE_INSTALL_PREFIX=${libcudf.install.path}"/>
       <arg value="-DCUDA_STATIC_RUNTIME=ON"/>
       <arg value="-DCUDF_ENABLE_ARROW_S3=OFF"/>
-      <arg value="-DCUDF_ENABLE_ARROW_PARQUET=OF"/>
+      <arg value="-DCUDF_ENABLE_ARROW_PARQUET=ON"/>
       <arg value="-DCUDF_USE_ARROW_STATIC=ON"/>
       <arg value="-DPER_THREAD_DEFAULT_STREAM=${PER_THREAD_DEFAULT_STREAM}" />
       <arg value="-DRMM_LOGGING_LEVEL=${RMM_LOGGING_LEVEL}" />

diff --git a/build/Dockerfile.centos7 b/build/Dockerfile.centos7
diff --git a/build/build-in-docker b/build/build-in-docker
@@ -31,7 +31,6 @@ LOCAL_MAVEN_REPO=${LOCAL_MAVEN_REPO:-"$HOME/.m2/repository"}
 PER_THREAD_DEFAULT_STREAM=${PER_THREAD_DEFAULT_STREAM:-ON}
 USE_GDS=${USE_GDS:-ON}
 
-CUDF_IMAGE_NAME="cudf-build:${CUDA_VERSION}-devel-centos7"
 SPARK_IMAGE_NAME="spark-rapids-jni-build:${CUDA_VERSION}-devel-centos7"
 WORKSPACE_DIR=/rapids
 WORKSPACE_REPODIR="$WORKSPACE_DIR/spark-rapids-jni"
@@ -42,12 +41,7 @@ if (( $# == 0 )); then
   exit 1
 fi
 
-$DOCKER_CMD build -f $REPODIR/thirdparty/cudf/java/ci/Dockerfile.centos7 \
-  --build-arg CUDA_VERSION=$CUDA_VERSION \
-  -t $CUDF_IMAGE_NAME \
-  $REPODIR/thirdparty/cudf/java/ci
-
-$DOCKER_CMD build -f $REPODIR/build/Dockerfile.centos7 \
+$DOCKER_CMD build -f $REPODIR/ci/Dockerfile \
   --build-arg CUDA_VERSION=$CUDA_VERSION \
   -t $SPARK_IMAGE_NAME \
   $REPODIR/build