Skip to content

Conversation

@zhangshenghang
Copy link
Member

Purpose of this pull request

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

@zhangshenghang zhangshenghang changed the title [Feature] Version from 2.3.10 to 2.3.12 [Feature] Version from 2.3.11 to 2.3.12 May 27, 2025
@github-actions github-actions bot added document dependencies Pull requests that update a dependency file labels May 27, 2025
@nielifeng nielifeng requested a review from Copilot May 27, 2025 06:40
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR bumps SeaTunnel’s version identifiers from 2.3.11 to 2.3.12 across dependencies, build configuration, and documentation.

  • Updated snapshot artifact versions and revision property for the 2.3.12 release
  • Adjusted all localized and English docs to reference the new 2.3.12 version

Reviewed Changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
tools/dependencies/known-dependencies.txt Bumped optional SeaTunnel jar snapshots to 2.3.12
pom.xml Updated <revision> to 2.3.12-SNAPSHOT
docs/zh/start-v2/locally/deployment.md Changed download and install-plugin version to 2.3.12
docs/zh/start-v2/docker/docker.md Updated Maven and Docker tags to 2.3.12
docs/zh/seatunnel-engine/download-seatunnel.md Bumped plugin install script version to 2.3.12
docs/en/start-v2/locally/deployment.md Changed download and install-plugin version to 2.3.12
docs/en/start-v2/kubernetes/kubernetes.mdx Updated environment and image tags to 2.3.12
docs/en/start-v2/docker/docker.md Updated Maven and Docker tags to 2.3.12
docs/en/seatunnel-engine/download-seatunnel.md Bumped plugin install script version to 2.3.12
Comments suppressed due to low confidence (2)

docs/zh/start-v2/docker/docker.md:42

  • [nitpick] This block originally referenced version 2.3.10 rather than 2.3.11; please verify that the earlier bump was applied correctly and align this example with the English docs.
sh bin/install-plugin.sh 2.3.12

docs/en/start-v2/docker/docker.md:42

  • [nitpick] You could introduce a variable (e.g. DOCKER_TAG) for the version in these Maven and Docker commands to avoid manually updating every occurrence on future releases.
sh ./mvnw -B clean install ... -D"docker.tag"=2.3.12 ...

Comment on lines +59 to +65
docker build -t seatunnel:2.3.12-flink-1.13 -f Dockerfile .
```
Image `seatunnel:2.3.11-flink-1.13` needs to be present in the host (minikube) so that the deployment can take place.
Image `seatunnel:2.3.12-flink-1.13` needs to be present in the host (minikube) so that the deployment can take place.

Load image to minikube via:
```bash
minikube image load seatunnel:2.3.11-flink-1.13
minikube image load seatunnel:2.3.12-flink-1.13
Copy link

Copilot AI May 27, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[nitpick] Consider using the ${SEATUNNEL_VERSION} environment variable in the Docker build and load commands instead of hard-coding the version to simplify future version bumps.

Copilot uses AI. Check for mistakes.
liunaijie
liunaijie previously approved these changes May 28, 2025
@Hisoka-X
Copy link
Member

wating test case passes.

@davidzollo
Copy link
Contributor

image

@zhangshenghang
Copy link
Member Author

zhangshenghang commented Jun 3, 2025 via email

@corgy-w corgy-w merged commit 14bf9d2 into apache:dev Jun 6, 2025
5 of 6 checks passed
dybyte pushed a commit to dybyte/seatunnel that referenced this pull request Jul 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved dependencies Pull requests that update a dependency file document reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants