Skip to content

Minor cleanup for Kerberized Hadoop KMS Environment.#13700

Merged
Praveen2112 merged 2 commits intotrinodb:masterfrom
Praveen2112:praveen/hive_krb_cleanup
Aug 19, 2022
Merged

Minor cleanup for Kerberized Hadoop KMS Environment.#13700
Praveen2112 merged 2 commits intotrinodb:masterfrom
Praveen2112:praveen/hive_krb_cleanup

Conversation

@Praveen2112
Copy link
Copy Markdown
Member

@Praveen2112 Praveen2112 commented Aug 17, 2022

Description

Minor code cleanup for Hadoop and Kerberized Hadoop KMS environment

Is this change a fix, improvement, new feature, refactoring, or other?

Refactoring

Is this a change to the core query engine, a connector, client library, or the SPI interfaces? (be specific)

For testing

How would you describe this change to a non-technical end user or system administrator?

Not required.

Related issues, pull requests, and links

Documentation

(x) No documentation is needed.
( ) Sufficient documentation is included in this PR.
( ) Documentation PR is available with #prnumber.
( ) Documentation issue #issuenumber is filed, and can be handled later.

Release notes

(x) No release notes entries required.
( ) Release notes entries required with the following suggested text:

# Section
* Fix some things. ({issue}`issuenumber`)

@cla-bot cla-bot bot added the cla-signed label Aug 17, 2022
@Praveen2112 Praveen2112 force-pushed the praveen/hive_krb_cleanup branch from 47fe91a to 4e6439a Compare August 17, 2022 09:16
@Praveen2112 Praveen2112 changed the title Minor cleanup for Kerberized Hadoop Environment. Minor cleanup for Kerberized Hadoop KMS Environment. Aug 17, 2022
@Praveen2112 Praveen2112 force-pushed the praveen/hive_krb_cleanup branch from 4e6439a to 4309d36 Compare August 17, 2022 12:19
Copy link
Copy Markdown
Member

@s2lomon s2lomon left a comment

Choose a reason for hiding this comment

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

lgtm

@Praveen2112 Praveen2112 merged commit 7c2f0c2 into trinodb:master Aug 19, 2022
@github-actions github-actions bot added this to the 394 milestone Aug 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

4 participants