Skip to content

[ZEPPELIN-5176] Kerberos ticket renewal fix in JDBC Interpreter#4001

Closed
VipinRathor wants to merge 1 commit into
apache:masterfrom
VipinRathor:ZEPPELIN-5176
Closed

[ZEPPELIN-5176] Kerberos ticket renewal fix in JDBC Interpreter#4001
VipinRathor wants to merge 1 commit into
apache:masterfrom
VipinRathor:ZEPPELIN-5176

Conversation

@VipinRathor
Copy link
Copy Markdown
Contributor

What is this PR for?

This PR addresses a problem in JDBC Interpreter where the Kerberos ticket renewal thread and the interpret() function vies for Kerberos ticket during Interpreter initialization and later on, during the ticket renewal, which can lead to an expired ticket intermittently.
This PR fixes Kerberos ticket renewal logic in JDBC interpreter:

  1. Initialize UGI before using it in runKerberosLogin()
  2. Wait 1 second between every call to runKerberosLogin()

What type of PR is it?

[Bug Fix]

Todos

  • none

What is the Jira issue?

How should this be tested?

  • Manual testing on a cluster. Configure JDBC interpreter with Kerberos enabled and make sure that the ticket is valid after the expiration time

Screenshots (if appropriate)

Questions:

  • Does the licenses files need update? No
  • Is there breaking changes for older versions? No
  • Does this needs documentation? No

@VipinRathor
Copy link
Copy Markdown
Contributor Author

@prabhjyotsingh / @Leemoonsoo / @PrarthiJain , please help review. Thanks.

Copy link
Copy Markdown
Contributor

@prabhjyotsingh prabhjyotsingh left a comment

Choose a reason for hiding this comment

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

LGTM

Fixing Kerberos ticket renewal logic in JDBC interpreter:
1. Initialize UGI before using
2. Wait 1 second between every call to runKerberosLogin()
@VipinRathor
Copy link
Copy Markdown
Contributor Author

Rebased after RAT check hotfix.

@prabhjyotsingh
Copy link
Copy Markdown
Contributor

Will merge this if no more discussion

@asfgit asfgit closed this in f188b40 Dec 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants