Skip to content

fix(iceberg): fix empty metastore password for iceberg engine. #22302

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jun 20, 2025
Merged

fix(iceberg): fix empty metastore password for iceberg engine. #22302

merged 2 commits into from
Jun 20, 2025
+11 −7

Conversation

chenzl25
Copy link
Contributor

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.

What's changed and what's your intention?

Checklist

  • I have written necessary rustdoc comments.
  • I have added necessary unit tests and integration tests.
  • I have added test labels as necessary.
  • I have added fuzzing tests or opened an issue to track them.
  • My PR contains breaking changes.
  • My PR changes performance-critical code, so I will run (micro) benchmarks and present the results.
  • I have checked the Release Timeline and Currently Supported Versions to determine which release branches I need to cherry-pick this PR into.

Documentation

  • My PR needs documentation updates.
Release note

@chenzl25 chenzl25 requested review from xiangjinwu and xxchan June 20, 2025 06:31
@github-actions github-actions bot added type/fix Type: Bug fix. Only for pull requests. ci/run-e2e-iceberg-tests labels Jun 20, 2025
@chenzl25 chenzl25 requested a review from yezizp2012 June 20, 2025 06:31
@chenzl25 chenzl25 enabled auto-merge June 20, 2025 06:39
@chenzl25 chenzl25 added this pull request to the merge queue Jun 20, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 20, 2025
@chenzl25 chenzl25 added this pull request to the merge queue Jun 20, 2025
Merged via the queue into main with commit baf8532 Jun 20, 2025
31 of 32 checks passed
@chenzl25 chenzl25 deleted the dylan/fix_iceberg_engine_empty_meta_password branch June 20, 2025 09:35
github-actions bot pushed a commit that referenced this pull request Jun 20, 2025
@chenzl25
fix(iceberg): fix empty metastore password for iceberg engine. (#22302)
6610731
@risingwave-ci risingwave-ci mentioned this pull request Jun 20, 2025
Merged
@github-actions GitHub Actions
Copy link
Contributor

✅ Cherry-pick PRs (or issues if encountered conflicts) have been created successfully to all target branches.

github-merge-queue bot pushed a commit that referenced this pull request Jun 20, 2025
@risingwave-ci @chenzl25
fix(iceberg): fix empty metastore password for iceberg engine. (#22302)…
26b6557 
… (#22308)

Co-authored-by: Dylan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yezizp2012 yezizp2012 yezizp2012 approved these changes

@xiangjinwu xiangjinwu Awaiting requested review from xiangjinwu

@xxchan xxchan Awaiting requested review from xxchan

Assignees
No one assigned
Labels
ci/run-e2e-iceberg-tests need-cherry-pick-since-release-2.4 type/fix Type: Bug fix. Only for pull requests.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Failed to parse password from meta store endpoint - when creating an table with the iceberg engine
2 participants
@chenzl25 @yezizp2012