From ce99c9ff1efbdaecb3a23d9e9002be0d2d639759 Mon Sep 17 00:00:00 2001
From: Gera Shegalov <gera@apache.org>
Date: Fri, 6 May 2022 19:52:59 -0700
Subject: [PATCH] Revert "Use ccache in build-in-docker by default" (#227)

* Revert "Use ccache in build-in-docker by default (#225)"

This reverts commit 1a0689c527ed839b54ef92ab2a0df96ba539a1b8.

* signing the change

Signed-off-by: Gera Shegalov <gera@apache.org>
---
 CONTRIBUTING.md       |  9 +++------
 build-libcudf.xml     |  1 -
 build/build-in-docker | 23 -----------------------
 pom.xml               |  2 --
 4 files changed, 3 insertions(+), 32 deletions(-)

diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
index d359f79236..0222e04b90 100644
--- a/CONTRIBUTING.md
+++ b/CONTRIBUTING.md
@@ -49,12 +49,9 @@ so it should be invoked as one would invoke Maven, e.g.: `build/build-in-docker
 ### cudf Submodule and Build
 
 [RAPIDS cuDF](https://github.com/rapidsai/cudf) is being used as a submodule in this project.
-Due to the lengthy build of libcudf, it is **not cleaned** during a normal Maven clean phase
-unless built using `build/build-in-docker`. `build/build-in-docker` uses `ccache` by default
-unless CCACHE_DISABLE=1 is set in the environment.
-
-`-Dlibcudf.clean.skip=false` can also be specified on the Maven command-line to force
-libcudf to be cleaned during the Maven clean phase.
+Due to the lengthy build of libcudf, it is **not cleaned** during a normal Maven clean phase.
+Use `-Dlibcudf.clean.skip=true` to clean the libcudf build area in addition to the normal clean
+of `target/` directories.
 
 Currently libcudf is only configured once and the build relies on cmake to re-configure as needed.
 This is because libcudf currently is rebuilding almost entirely when it is configured with the same
diff --git a/build-libcudf.xml b/build-libcudf.xml
index d4eb73c598..81ffb21c0e 100644
--- a/build-libcudf.xml
+++ b/build-libcudf.xml
@@ -36,7 +36,6 @@
           executable="cmake"
           if:true="${needConfigure}">
       <arg value="${cudf.path}/cpp"/>
-      <arg line="${cmake.ccache.opts}"/>
       <arg value="-DBUILD_SHARED_LIBS=OFF"/>
       <arg value="-DBUILD_TESTS=OFF"/>
       <arg value="-DCMAKE_CUDA_ARCHITECTURES=${GPU_ARCHS}"/>
diff --git a/build/build-in-docker b/build/build-in-docker
index c52d5f31fe..462dfdad3b 100755
--- a/build/build-in-docker
+++ b/build/build-in-docker
@@ -46,25 +46,6 @@ $DOCKER_CMD build -f $REPODIR/ci/Dockerfile \
   -t $SPARK_IMAGE_NAME \
   $REPODIR/build
 
-CCACHE_OPTS=(
-  "-DCMAKE_C_COMPILER_LAUNCHER=ccache"
-  "-DCMAKE_CXX_COMPILER_LAUNCHER=ccache"
-  "-DCMAKE_CUDA_COMPILER_LAUNCHER=ccache"
-  "-DCMAKE_CXX_LINKER_LAUNCHER=ccache"
-)
-
-_CUDF_CLEAN_SKIP=""
-# if ccache is enabled and libcudf.clean.skip not provided
-# by the user remove the cpp build directory
-#
-if [[ "$CCACHE_DISABLE" != "1" ]]; then
-  if [[ ! "$*" =~ " -Dlibcudf.clean.skip=" ]]; then
-    # Don't skip clean if ccache is enabled
-    # unless the user overrides
-    _CUDF_CLEAN_SKIP="-Dlibcudf.clean.skip=false"
-  fi
-fi
-
 $DOCKER_CMD run -it -u $(id -u):$(id -g) --rm \
   -v "/etc/group:/etc/group:ro" \
   -v "/etc/passwd:/etc/passwd:ro" \
@@ -72,9 +53,7 @@ $DOCKER_CMD run -it -u $(id -u):$(id -g) --rm \
   -v "/etc/sudoers.d:/etc/sudoers.d:ro" \
   -v "$REPODIR:$WORKSPACE_REPODIR:rw" \
   -v "$LOCAL_MAVEN_REPO:$WORKSPACE_MAVEN_REPODIR:rw" \
-  -v "$HOME/.ccache:$HOME/.ccache:rw" \
   --workdir "$WORKSPACE_REPODIR" \
-  -e CCACHE_DISABLE \
   -e CMAKE_GENERATOR="$CMAKE_GENERATOR" \
   -e CUDA_VISIBLE_DEVICES \
   -e PARALLEL_LEVEL \
@@ -84,6 +63,4 @@ $DOCKER_CMD run -it -u $(id -u):$(id -g) --rm \
     -Dmaven.repo.local=$WORKSPACE_MAVEN_REPODIR \
     -DPER_THREAD_DEFAULT_STREAM=$PER_THREAD_DEFAULT_STREAM \
     -DUSE_GDS=$USE_GDS \
-    -Dcmake.ccache.opts=\"${CCACHE_OPTS[*]}\" \
-    $_CUDF_CLEAN_SKIP \
     $*"
diff --git a/pom.xml b/pom.xml
index 5cbd892188..8a17befb02 100644
--- a/pom.xml
+++ b/pom.xml
@@ -100,7 +100,6 @@
     <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
     <slf4j.version>1.7.30</slf4j.version>
     <submodule.check.skip>false</submodule.check.skip>
-    <cmake.ccache.opts/>
   </properties>
 
   <dependencies>
@@ -261,7 +260,6 @@
                   <env key="CUDF_CPP_BUILD_DIR" value="${libcudf.build.path}"/>
                   <env key="CUDF_ROOT" value="${cudf.path}"/>
                   <arg value="${cudf.path}/java/src/main/native"/>
-                  <arg line="${cmake.ccache.opts}"/>
                   <arg value="-DBUILD_SHARED_LIBS=OFF"/>
                   <arg value="-DCUDA_STATIC_RUNTIME=ON"/>
                   <arg value="-DCUDF_JNI_LIBCUDF_STATIC=ON"/>