Skip to content

Commit 1e533c8

Browse files
committed
[native] Fix dependency build dockerfiles
1 parent d3f7172 commit 1e533c8

File tree

4 files changed

+7
-7
lines changed

4 files changed

+7
-7
lines changed

.github/workflows/prestocpp-linux-adapters-build.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ jobs:
4040
cd presto-native-execution
4141
export DEPENDENCY_DIR=${GITHUB_WORKSPACE}/adapter-deps/download
4242
export INSTALL_PREFIX=${GITHUB_WORKSPACE}/adapter-deps/install
43-
PROMPT_ALWAYS_RESPOND=n ./velox/scripts/setup-adapters.sh
43+
PROMPT_ALWAYS_RESPOND=n ./velox/scripts/setup-centos9.sh install_adapters
4444
PROMPT_ALWAYS_RESPOND=n ./scripts/setup-adapters.sh
4545
4646
- name: Install Github CLI for using apache/infrastructure-actions/stash

presto-native-execution/README.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -84,10 +84,10 @@ To enable Parquet and S3 support, set `PRESTO_ENABLE_PARQUET = "ON"`,
8484
`PRESTO_ENABLE_S3 = "ON"` in the environment.
8585

8686
S3 support needs the [AWS SDK C++](https://github.com/aws/aws-sdk-cpp) library.
87-
This dependency can be installed by running the script below from the
88-
`presto/presto-native-execution` directory.
87+
This dependency can be installed by running the target platform build script
88+
from the `presto/presto-native-execution` directory.
8989

90-
`./velox/scripts/setup-adapters.sh aws`
90+
`./velox/scripts/setup-centos9.sh install_aws`
9191

9292
#### JWT Authentication
9393
To enable JWT authentication support, set `PRESTO_ENABLE_JWT = "ON"` in

presto-native-execution/scripts/dockerfiles/centos-dependency.dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ COPY scripts /scripts
2121
COPY velox/scripts /velox/scripts
2222
RUN mkdir build && \
2323
(cd build && ../scripts/setup-centos.sh && \
24-
../velox/scripts/setup-adapters.sh && \
24+
../velox/scripts/setup-centos9.sh install_adapters && \
2525
../scripts/setup-adapters.sh && \
2626
../velox/scripts/setup-centos9.sh install_clang15) && \
2727
rm -rf build

presto-native-execution/scripts/dockerfiles/ubuntu-22.04-dependency.dockerfile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,10 +26,10 @@ ENV TZ=${tz}
2626
RUN mkdir -p /scripts /velox/scripts
2727
COPY scripts /scripts
2828
COPY velox/scripts /velox/scripts
29-
# setup-adapters.sh does not install rpm needed for minio install.
29+
# install rpm needed for minio install.
3030
RUN mkdir build && \
3131
(cd build && ../scripts/setup-ubuntu.sh && \
3232
apt install -y rpm && \
33-
../velox/scripts/setup-adapters.sh && \
33+
../velox/scripts/setup-ubuntu.sh install_adapters && \
3434
../scripts/setup-adapters.sh ) && \
3535
rm -rf build

0 commit comments

Comments
 (0)