[VL] Fix compression type 2 issue

apache · Mar 25, 2024 · b0d9e58 · b0d9e58
1 parent e00f051
commit b0d9e58
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/.github/workflows/velox_docker.yml b/.github/workflows/velox_docker.yml
@@ -50,7 +50,7 @@ jobs:
       - uses: actions/checkout@v2
       - name: Build Gluten velox third party
         run: |
-          yum install sudo patch java-1.8.0-openjdk-devel -y && \
+          yum install sudo patch dnf java-1.8.0-openjdk-devel -y && \
           cd $GITHUB_WORKSPACE/ep/build-velox/src && \
           ./get_velox.sh && \
           source /opt/rh/devtoolset-9/enable && \

diff --git a/dev/vcpkg/ports/folly/fix-deps.patch b/dev/vcpkg/ports/folly/fix-deps.patch
@@ -128,6 +128,7 @@ index 4b78e9f..ac83c99 100644
 +if (NOT CMAKE_DISABLE_FIND_PACKAGE_LZ4)
 +  find_package(lz4 CONFIG)
 +  if(TARGET lz4::lz4)
++    set(FOLLY_HAVE_LIBLZ4 1)
 +    list(APPEND FOLLY_LINK_LIBRARIES lz4::lz4)
 +  endif()
  endif()

diff --git a/ep/build-velox/src/get_velox.sh b/ep/build-velox/src/get_velox.sh
@@ -170,8 +170,8 @@ function process_setup_centos7 {
   # No need to re-install git.
   sed -i 's/dnf_install ccache git/dnf_install ccache/' scripts/setup-centos7.sh
   # cmake 3 and ninja should be installed
-  sed -i '/^run_and_time install_cmake/d' scripts/setup-centos7.sh
-  sed -i '/^run_and_time install_ninja/d' scripts/setup-centos7.sh
+  # sed -i '/^run_and_time install_cmake/d' scripts/setup-centos7.sh
+  # sed -i '/^run_and_time install_ninja/d' scripts/setup-centos7.sh
 
   # install gtest
   sed -i '/^  run_and_time install_folly/a \ \ run_and_time install_gtest' scripts/setup-centos7.sh