Skip to content

Commit

Permalink
sed install_awssdk function to setup-centos8.sh#velox on centos8
Browse files Browse the repository at this point in the history
  • Loading branch information
dcoliversun authored and hengzhen.sq committed Nov 3, 2023
1 parent e597c59 commit 798e4a1
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/velox_be.yml
Original file line number Diff line number Diff line change
Expand Up @@ -354,7 +354,7 @@ jobs:
source /env.sh && \
sudo yum -y install patch && \
cd /opt/gluten/ep/build-velox/src && \
./get_velox.sh --velox_home=/opt/velox --enable_hdfs=ON && \
./get_velox.sh --velox_home=/opt/velox --enable_hdfs=ON --enable_s3=ON && \
./build_velox.sh --velox_home=/opt/velox --enable_ep_cache=ON --enable_hdfs=ON'
- name: Build Gluten CPP library
run: |
Expand Down
1 change: 1 addition & 0 deletions ep/build-velox/src/get_velox.sh
Original file line number Diff line number Diff line change
Expand Up @@ -112,6 +112,7 @@ function process_setup_centos8 {
sed -i '/^cmake_install_deps fmt/a \\install_protobuf' scripts/setup-centos8.sh
fi
if [ $ENABLE_S3 == "ON" ]; then
sed -i '/^function cmake_install_deps.*/i function install_awssdk {\n github_checkout aws/aws-sdk-cpp 1.9.379 --depth 1 --recurse-submodules\n cmake_install -DCMAKE_BUILD_TYPE=Release -DBUILD_SHARED_LIBS:BOOL=OFF -DMINIMIZE_SIZE:BOOL=ON -DENABLE_TESTING:BOOL=OFF -DBUILD_ONLY:STRING="s3;identity-management" \n} \n' scripts/setup-centos8.sh
sed -i '/^cmake_install_deps fmt/a \ \ install_awssdk' scripts/setup-centos8.sh
fi
}
Expand Down

0 comments on commit 798e4a1

Please sign in to comment.