-
Notifications
You must be signed in to change notification settings - Fork 2.2k
[Feature] Version from 2.3.11 to 2.3.12 #9366
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this 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 ...
| 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 |
Copilot
AI
May 27, 2025
There was a problem hiding this comment.
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.
|
wating test case passes. |
|
thks. I will handle it recently
…---Original---
From: "David ***@***.***>
Date: Tue, Jun 3, 2025 18:10 PM
To: ***@***.***>;
Cc: ***@***.******@***.***>;
Subject: Re: [apache/seatunnel] [Feature] Version from 2.3.11 to 2.3.12 (PR#9366)
davidzollo left a comment (apache/seatunnel#9366)
image.png (view on web)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
2394d33 to
a3e8891
Compare

Purpose of this pull request
Does this PR introduce any user-facing change?
How was this patch tested?
Check list
New License Guide