Skip to content

[build][release] Improve version handling in deploy_staging_jars.sh#4335

Merged
lvyanquan merged 1 commit intoapache:masterfrom
lvyanquan:build
Mar 25, 2026
Merged

[build][release] Improve version handling in deploy_staging_jars.sh#4335
lvyanquan merged 1 commit intoapache:masterfrom
lvyanquan:build

Conversation

@lvyanquan
Copy link
Contributor

Background

Modify the revision property so that the package built by flink-cdc-dist includes FLINK_VERSION information:

<finalName>${project.artifactId}-${revision}</finalName>

Usage

RELEASE_VERSION=${RELEASE_VERSION} FLINK_VERSION=${FLINK_VERSION} ./tools/releasing/deploy_staging_jars.sh

- Add set_revision_property function to update Maven revision property
- Require RELEASE_VERSION environment variable for explicit version control
- Update version format to RELEASE_VERSION-FLINK_VERSION pattern
- Set both pom version and revision property for consistency

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions github-actions bot added the build label Mar 24, 2026
@lvyanquan
Copy link
Contributor Author

PTAL @yuxiqian.

Copy link
Member

@yuxiqian yuxiqian left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@lvyanquan lvyanquan merged commit 89d6f93 into apache:master Mar 25, 2026
53 of 56 checks passed
lvyanquan added a commit that referenced this pull request Mar 25, 2026
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
(cherry picked from commit 89d6f93)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants