Skip to content

Docs: Update statements mentioning Hive's alpha/beta versions - #12430

Merged
pvary merged 2 commits into
apache:mainfrom
okumin:hive-version
Mar 19, 2025
Merged

pvary merged 2 commits into
apache:mainfrom
okumin:hive-version

Conversation

@okumin

@okumin okumin commented Mar 1, 2025 •

Copy link
Copy Markdown
Contributor

Our doc mentions 4.0.0-alpha-1, 4.0.0-alpha-2, and 4.0.0-beta-1, maybe because the Hive community had struggled to release 4.0.0. Now that it's been approximately 1 year since we released 4.0.0, I think we don't need to mention all the RC versions in our document.
https://hive.apache.org/general/downloads/

@github-actions github-actions Bot added the docs label Mar 1, 2025
Comment thread docs/docs/hive.md Outdated
### Hive 4.0.0-alpha-1

Hive 4.0.0-alpha-1 comes with the Iceberg 0.13.1 included.
Hive 4.0.x comes with the Iceberg 1.4.3 included.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nit: can we remove 'the' in front of Iceberg while we are here. Its not common to have the article before Iceberg.

@pvary

pvary commented Mar 2, 2025

Copy link
Copy Markdown
Contributor

@deniskuzZ: Could you please review?

@gaborkaszab gaborkaszab left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Seems good to me. Thanks!

@pvary
pvary merged commit 8204462 into apache:main Mar 19, 2025
@okumin
okumin deleted the hive-version branch March 19, 2025 12:49
@okumin

okumin commented Mar 19, 2025

Copy link
Copy Markdown
Contributor Author

Thanks for merging!

@deniskuzZ

Copy link
Copy Markdown
Member

hi @pvary, sorry, I missed the notification. Thx for merging!

@pvary

pvary commented Mar 19, 2025

Copy link
Copy Markdown
Contributor

Thanks for the PR and review everyone!

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants