Skip to content

Conversation

@gh-yzou
Copy link
Contributor

@gh-yzou gh-yzou commented Nov 19, 2025

Remove 'beta' label for Generic Table

Checklist

  • 🛡️ Don't disclose security issues! (contact security@apache.org)
  • 🔗 Clearly explained why the changes are needed, or linked related issues: Fixes #
  • 🧪 Added/updated tests with good coverage, or manually tested (and explained how)
  • 💡 Added comments for complex logic
  • 🧾 Updated CHANGELOG.md (if needed)
  • 📚 Updated documentation in site/content/in-dev/unreleased (if needed)

jbonofre
jbonofre previously approved these changes Nov 19, 2025
@github-project-automation github-project-automation bot moved this from PRs In Progress to Ready to merge in Basic Kanban Board Nov 19, 2025
dimas-b
dimas-b previously approved these changes Nov 19, 2025
CHANGELOG.md Outdated
- `client.region` is no longer considered a "credential" property (related to Iceberg REST Catalog API).
- Relaxed the requirements for S3 storage's ARN to allow Polaris to connect to more non-AWS S3 storage appliances.
- Added checksum to helm deployment so that it will restart when the configmap has changed.
- Removed beta label for Generic Table.
Copy link
Contributor

Choose a reason for hiding this comment

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

To me, I believe that "removing the beta label" undersells what we are doing here. Personally, I would rephrase as "Generic Tables are no longer in beta and are generally-available."

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sg! updated

@gh-yzou gh-yzou dismissed stale reviews from dimas-b and jbonofre via 8e36fda November 19, 2025 19:12
dimas-b
dimas-b previously approved these changes Nov 19, 2025
HonahX
HonahX previously approved these changes Nov 20, 2025
pingtimeout
pingtimeout previously approved these changes Nov 20, 2025
flyrain
flyrain previously approved these changes Nov 20, 2025
@pingtimeout pingtimeout dismissed stale reviews from flyrain, HonahX, dimas-b, and themself via 3795169 November 21, 2025 17:30
@pingtimeout
Copy link
Contributor

Fixed the conflict on the changelog

@gh-yzou gh-yzou merged commit d5588c4 into main Nov 21, 2025
14 of 16 checks passed
@gh-yzou gh-yzou deleted the yzou-remove-generic-table-beta branch November 21, 2025 23:01
@github-project-automation github-project-automation bot moved this from Ready to merge to Done in Basic Kanban Board Nov 21, 2025
@jbonofre
Copy link
Member

@gh-yzou next time, please, before merging, be sure we have a consensus within the community (c.f. my reply on the dev@ mailing list).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants