This is an automated email from the ASF dual-hosted git repository.

yuqi1129 pushed a commit to branch branch-1.3
in repository https://gitbox.apache.org/repos/asf/gravitino.git


The following commit(s) were added to refs/heads/branch-1.3 by this push:
     new e0c212f3da [Cherry-pick to branch-1.3] [#13461] 
improvement(connector): Bundle connector image README into Trino/Flink/Spark 
images (#13470) (#13479)
e0c212f3da is described below

commit e0c212f3dad26294d4d9c00ef4bdaf011c5f203b
Author: geyanggang <[email protected]>
AuthorDate: Wed Sep 23 22:03:19 2026 +0800

    [Cherry-pick to branch-1.3] [#13461] improvement(connector): Bundle 
connector image README into Trino/Flink/Spark images (#13470) (#13479)
    
    ### What changes were proposed in this pull request?
    
    Cherry-pick of #13470 to `branch-1.3`.
    
    Adds `COPY README.md /README.md` to the three connector image
    Dockerfiles so the
    connector-image usage documentation ships inside the image:
    
    - `dev/docker/trino-connectors/Dockerfile`
    - `dev/docker/flink-connectors/Dockerfile`
    - `dev/docker/spark-connectors/Dockerfile`
    
    Existing `/licenses` handling and the Trino version-band
    `LICENSE`/`NOTICE`/`README.md` under `/connectors/trino-*/` are
    unchanged. The
    new `/README.md` is at a different path and does not conflict with the
    band
    README.
    
    ### Why are the changes needed?
    
    The connector images only ship the connector packages, the entrypoint
    and
    `/licenses`; the image usage README is not included, so users cannot
    read
    install/usage docs from the image itself.
    
    Fix: #13461
    
    ### Does this PR introduce _any_ user-facing change?
    
    Yes. Users can read the usage documentation from the image, e.g.
    `docker run --rm --entrypoint cat <image> /README.md`.
    
    ### How was this patch tested?
    
    Clean cherry-pick of #13470 (no conflicts). Verified the `COPY README.md
    /README.md`
    line is present in all three Dockerfiles and that each build context
    contains
    `README.md`.
---
 dev/docker/flink-connectors/Dockerfile | 3 +++
 dev/docker/spark-connectors/Dockerfile | 3 +++
 dev/docker/trino-connectors/Dockerfile | 3 +++
 3 files changed, 9 insertions(+)

diff --git a/dev/docker/flink-connectors/Dockerfile 
b/dev/docker/flink-connectors/Dockerfile
index 5448554747..d25a430d81 100644
--- a/dev/docker/flink-connectors/Dockerfile
+++ b/dev/docker/flink-connectors/Dockerfile
@@ -73,6 +73,9 @@ COPY --chmod=755 copy-connector.sh /copy-connector.sh
 # --- licenses directory (LICENSE + NOTICE + third-party summary) ---
 COPY licenses /licenses
 
+# --- Connector image usage documentation ---
+COPY README.md /README.md
+
 # --- Drop source-control artifacts, OpenShift arbitrary UID compatibility,
 #     and a passwd entry for UID 1000 ---
 RUN rm -f /licenses/.gitignore \
diff --git a/dev/docker/spark-connectors/Dockerfile 
b/dev/docker/spark-connectors/Dockerfile
index 395e26a412..9960924952 100644
--- a/dev/docker/spark-connectors/Dockerfile
+++ b/dev/docker/spark-connectors/Dockerfile
@@ -75,6 +75,9 @@ COPY --chmod=755 copy-connector.sh /copy-connector.sh
 # --- licenses directory (LICENSE + NOTICE + third-party summary) ---
 COPY licenses /licenses
 
+# --- Connector image usage documentation ---
+COPY README.md /README.md
+
 # --- Drop source-control artifacts, OpenShift arbitrary UID compatibility,
 #     and a passwd entry for UID 1000 ---
 RUN rm -f /licenses/.gitignore \
diff --git a/dev/docker/trino-connectors/Dockerfile 
b/dev/docker/trino-connectors/Dockerfile
index 11cfcbefa6..cf241f2bfd 100644
--- a/dev/docker/trino-connectors/Dockerfile
+++ b/dev/docker/trino-connectors/Dockerfile
@@ -73,6 +73,9 @@ COPY --chmod=755 copy-connector.sh /copy-connector.sh
 # --- licenses directory (LICENSE + NOTICE + third-party summary) ---
 COPY licenses /licenses
 
+# --- Connector image usage documentation ---
+COPY README.md /README.md
+
 # --- Drop source-control artifacts, OpenShift arbitrary UID compatibility,
 #     and a passwd entry for UID 1000 ---
 RUN rm -f /licenses/.gitignore \

Reply via email to