Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BugFix] Solve issues-3470: Yarn webui fails to obtain task status when submitting a Flink task after turning on Kerberos authentication. #4020

Merged
merged 10 commits into from
Dec 9, 2024

Conversation

miaoze8
Copy link
Contributor

@miaoze8 miaoze8 commented Dec 6, 2024

解决yarn webui开启Kerberos认证后递交Flink任务无法获取任务状态的问题,增加调用Flink HTTP API时候进行Kerberos认证,
相关issues: #3470

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.

(or)

This pull request is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

@github-actions github-actions bot changed the title 解决issues-3470 yarn webui开启Kerberos认证后递交Flink任务无法获取任务状态的问题 Solve issues-3470: Yarn webui fails to obtain task status when submitting a Flink task after turning on Kerberos authentication. Dec 6, 2024
@zackyoungh zackyoungh added the Bug Something isn't working label Dec 6, 2024
@zackyoungh zackyoungh added this to the 1.2.0 milestone Dec 6, 2024
@zackyoungh zackyoungh changed the title Solve issues-3470: Yarn webui fails to obtain task status when submitting a Flink task after turning on Kerberos authentication. [BugFix] Solve issues-3470: Yarn webui fails to obtain task status when submitting a Flink task after turning on Kerberos authentication. Dec 6, 2024
@zackyoungh
Copy link
Contributor

请使用spotless 格式化代码,详细查看此文:https://www.dinky.org.cn/docs/next/developer_guide/contribution/pull_request#pull-request-code-style

miaoze8

This comment was marked as duplicate.

Copy link
Contributor Author

@miaoze8 miaoze8 left a comment

Choose a reason for hiding this comment

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

已调整

Copy link
Contributor Author

@miaoze8 miaoze8 left a comment

Choose a reason for hiding this comment

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

调整 import x.*

Copy link
Contributor

@aiwenmo aiwenmo left a comment

Choose a reason for hiding this comment

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

Hi. Thank you for your contribution. I have a question.

Copy link
Contributor Author

@miaoze8 miaoze8 left a comment

Choose a reason for hiding this comment

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

更改判断方式,仅在获取flink任务状态时候HTTP如果返回401状态时,额外采用Kerberos方式认证HTTP获取flink任务状态

@miaoze8 miaoze8 requested a review from aiwenmo December 9, 2024 08:05
@miaoze8 miaoze8 requested a review from aiwenmo December 9, 2024 08:25
@aiwenmo
Copy link
Contributor

aiwenmo commented Dec 9, 2024

Hi. Run 'mvn spotless:apply' to fix these violations.

解决yarn webui开启Kerberos认证后递交Flink任务无法获取任务状态的问题,增加调用Flink HTTP API时候进行Kerberos认证,相关issues: DataLinkDC#3470
Copy link
Contributor Author

@miaoze8 miaoze8 left a comment

Choose a reason for hiding this comment

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

spotless:apply done

Copy link
Contributor

@aiwenmo aiwenmo left a comment

Choose a reason for hiding this comment

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

LGTM

@aiwenmo aiwenmo merged commit 4a2dfde into DataLinkDC:dev Dec 9, 2024
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
Status: Done
4 participants