Skip to content

Docs: Fix Flink Getting Started page#15772

Merged
pvary merged 3 commits intoapache:mainfrom
manuzhang:fix-flink-docs
Mar 27, 2026
Merged

Docs: Fix Flink Getting Started page#15772
pvary merged 3 commits intoapache:mainfrom
manuzhang:fix-flink-docs

Conversation

@manuzhang
Copy link
Copy Markdown
Member

@manuzhang manuzhang commented Mar 26, 2026

Update the Flink docs to use the correct Iceberg metadata-table syntax, clean up the generic CREATE CATALOG example, and replace hardcoded runtime jar examples with version-templated names. Also fix the PyFlink install snippet and several wording issues in the writing and reading sections.

Co-authored-by: @codex

@github-actions github-actions bot added the docs label Mar 26, 2026
@manuzhang manuzhang force-pushed the fix-flink-docs branch 2 times, most recently from 0e02af2 to f68132e Compare March 26, 2026 04:11
@manuzhang manuzhang changed the title Fix Flink docs examples Docs: Fix Getting Started page Mar 26, 2026
@manuzhang manuzhang requested a review from pvary March 26, 2026 04:13
Update the Flink docs to use the correct Iceberg metadata-table syntax, clean up the generic CREATE CATALOG example, and replace hardcoded runtime jar examples with version-templated names. Also fix the PyFlink install snippet and several wording issues in the writing and reading sections.

Co-authored-by: Codex <codex@openai.com>
@manuzhang manuzhang changed the title Docs: Fix Getting Started page Docs: Fix Flink Getting Started page Mar 26, 2026
@pvary
Copy link
Copy Markdown
Contributor

pvary commented Mar 26, 2026

CC: @mxm, @Guosmilesmile - any comments?

Copy link
Copy Markdown
Contributor

@Guosmilesmile Guosmilesmile left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Copy Markdown
Contributor

@mxm mxm left a comment

Choose a reason for hiding this comment

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

Looks a lot better. Thanks @manuzhang!

manuzhang and others added 2 commits March 27, 2026 17:37
Co-authored-by: Maximilian Michels <mxm@apache.org>
Co-authored-by: Maximilian Michels <mxm@apache.org>
@pvary pvary merged commit 9011eac into apache:main Mar 27, 2026
6 checks passed
@pvary
Copy link
Copy Markdown
Contributor

pvary commented Mar 27, 2026

Merged to main.
Thanks @manuzhang for the PR and everyone for the review!

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.

6 participants