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

Support Kerberos auth #2516

Merged
merged 1 commit into from
Nov 7, 2018
Merged

Support Kerberos auth #2516

merged 1 commit into from
Nov 7, 2018

Conversation

ofek
Copy link
Contributor

@ofek ofek commented Nov 6, 2018

Motivation

Customer requests

@codecov-io
Copy link

Codecov Report

Merging #2516 into master will decrease coverage by 7.69%.
The diff coverage is 60%.

@@            Coverage Diff             @@
##           master    #2516      +/-   ##
==========================================
- Coverage   83.84%   76.14%    -7.7%     
==========================================
  Files         640       51     -589     
  Lines       36448     3354   -33094     
  Branches     4440      393    -4047     
==========================================
- Hits        30559     2554   -28005     
+ Misses       4570      697    -3873     
+ Partials     1319      103    -1216

Copy link
Contributor

@zippolyte zippolyte left a comment

Choose a reason for hiding this comment

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

LGTM

@ofek ofek merged commit 0fece55 into master Nov 7, 2018
@ofek ofek deleted the ofek/hdfs branch November 7, 2018 20:09
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.

3 participants