Skip to content

[Improve][Docs] Iceberg connector adds parameter documentation related to Kerberos authentication#10704

Merged
zhangshenghang merged 5 commits into
apache:devfrom
chl-wxp:iceberg-add-kerberos-desc
Apr 8, 2026
Merged

[Improve][Docs] Iceberg connector adds parameter documentation related to Kerberos authentication#10704
zhangshenghang merged 5 commits into
apache:devfrom
chl-wxp:iceberg-add-kerberos-desc

Conversation

@chl-wxp
Copy link
Copy Markdown
Member

@chl-wxp chl-wxp commented Apr 3, 2026

Purpose of this pull request

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

Copy link
Copy Markdown
Contributor

@DanielLeens DanielLeens left a comment

Choose a reason for hiding this comment

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

I pulled this locally as a docs-only change and checked the added Iceberg Kerberos parameters against the current option names used in code.

From my side, the patch is doing the right thing: it is documenting existing Kerberos-related options in the Iceberg source/sink docs, and the EN / ZH docs are updated together.

I do not see a blocker here.

DanielLeens

This comment was marked as duplicate.

Copy link
Copy Markdown
Contributor

@davidzollo davidzollo left a comment

Choose a reason for hiding this comment

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

+1
Good job

Copy link
Copy Markdown
Member

@zhangshenghang zhangshenghang left a comment

Choose a reason for hiding this comment

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

+1

@zhangshenghang zhangshenghang merged commit 93bba1a into apache:dev Apr 8, 2026
6 checks passed
onceMisery pushed a commit to onceMisery/seatunnel that referenced this pull request Apr 14, 2026
junjunclub pushed a commit to junjunclub/seatunnel that referenced this pull request Apr 20, 2026
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.

4 participants