close
Skip to content

[Bug] Fix OrcWriteStrategy/ParquetWriteStrategy doesn't login with kerberos#6472

Merged
ruanwenjun merged 2 commits into
apache:devfrom
ruanwenjun:dev_wenjun_fixORC
Apr 8, 2024
Merged

[Bug] Fix OrcWriteStrategy/ParquetWriteStrategy doesn't login with kerberos#6472
ruanwenjun merged 2 commits into
apache:devfrom
ruanwenjun:dev_wenjun_fixORC

Conversation

@ruanwenjun

@ruanwenjun ruanwenjun commented Mar 8, 2024

Copy link
Copy Markdown
Member

Purpose of this pull request

When we use ORC/Parquet with kerberos in file connector or hive connector. we will get kerberos unauthorization exception, this is caused by we didn't login with kerberos in OrcWriteStrategy/ParquetWriteStrategy.

Does this PR introduce any user-facing change?

No

How was this patch tested?

Check list

@Hisoka-X Hisoka-X left a comment

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.

Shall we add test case?

@ruanwenjun

ruanwenjun commented Mar 11, 2024

Copy link
Copy Markdown
Member Author

Shall we add test case?

Yes, but this case will rely on Kerberos, I didn't find how to add UT.

@liugddx liugddx added this to the 2.3.5 milestone Mar 24, 2024
@hailin0

hailin0 commented Apr 8, 2024

Copy link
Copy Markdown
Member

Shall we add test case?

Yes, but this case will rely on Kerberos, I didn't find how to add UT.

It's okay, we can create an issue and complete it in the future to add test cases

@ruanwenjun

Copy link
Copy Markdown
Member Author

Shall we add test case?

Yes, but this case will rely on Kerberos, I didn't find how to add UT.

It's okay, we can create an issue and complete it in the future to add test cases

I create issue #6664 for this.

@hailin0

hailin0 commented Apr 8, 2024

Copy link
Copy Markdown
Member

link paimon sink kerberos

@hailin0 hailin0 left a comment

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.

LGTM

@ruanwenjun ruanwenjun merged commit 24441c8 into apache:dev Apr 8, 2024
@ruanwenjun ruanwenjun deleted the dev_wenjun_fixORC branch April 8, 2024 13:51
chaorongzhi pushed a commit to chaorongzhi/seatunnel that referenced this pull request Aug 21, 2024
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.

5 participants