This is an automated email from the ASF dual-hosted git repository.
yuqi4733 pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git
The following commit(s) were added to refs/heads/main by this push:
new 67d0cf5cb [#4688] fix (docker): Fix the Trino playground Docker image
cannot start up (#4689)
67d0cf5cb is described below
commit 67d0cf5cb99dcfef5ac1c4ff30807c81ca0c8f35
Author: Yuhui <[email protected]>
AuthorDate: Mon Aug 26 22:11:34 2024 +0800
[#4688] fix (docker): Fix the Trino playground Docker image cannot start up
(#4689)
### What changes were proposed in this pull request?
Fix the Trino playground Docker image cannot start up
### Why are the changes needed?
Fix: #4688
### Does this PR introduce _any_ user-facing change?
NO
### How was this patch tested?
Manually test
---
dev/docker/trino/Dockerfile | 14 ++++++++------
1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/dev/docker/trino/Dockerfile b/dev/docker/trino/Dockerfile
index 2050ab798..8d4e4ff3c 100644
--- a/dev/docker/trino/Dockerfile
+++ b/dev/docker/trino/Dockerfile
@@ -66,13 +66,15 @@ COPY --chown=trino:trino
packages/gravitino-trino-connector/postgresql-42.7.0.ja
RUN mkdir /tmp/gravitino
COPY --chown=trino:trino packages/gravitino-trino-connector /tmp/gravitino
-ARG IMAGE_NAME
ARG TAG_NAME
-RUN if [ "$TAG_NAME" = "trino-*" ] ; then \
- mv /tmp/gravitino /usr/lib/trino/plugin/; \
- else echo "Copying files for other images"; \
- fi
-
+RUN case "$TAG_NAME" in \
+ trino-*) \
+ mv /tmp/gravitino /usr/lib/trino/plugin/; \
+ ;; \
+ *) \
+ echo "Copying files for other images"; \
+ ;; \
+ esac
# The following ENV are needed to update configuration file before starting
Trino server, you need to
# pass these ENV variables to the container when starting it like `docker run
--env GRAVITINO_HOST_IP=xxx --env GRAVITINO_HOST_PORT=xxxx`