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

[IMPALA-12291] - impala checks hdfs ranger policy #54

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

HalimKim
Copy link
Contributor

Related Issue

Description

  • When HDFS is ranger enabled, current impala fe doesn't check ranger policy but hadoop inode permission
  • New feature adds getAvailableAccessLevel() method ranger check code using Hadoop API
  • Add one more catch in getTExecRequest() method to check AnalysisException

Test

  • Manual Test done
    • Distribute impala fe jar for impalad and catalogd
    • Enable HDFS Ranger Plugin.
    • Give impala Allow condition for ranger hdfs policy and check insert query - Insert Success
    • Give impala Deny condition for ranger hdfs policy and check insert query - Insert Fail

@MikaelSmith
Copy link
Contributor

Hi @HalimKim, thanks for your contribution! We don't use pull requests in Apache Impala, we use Gerrit instead: https://gerrit.cloudera.org/. https://cwiki.apache.org/confluence/display/IMPALA/Using+Gerrit+to+submit+and+review+patches has a walkthrough for uploading patches for review.

Thanks, Michael.

@HalimKim
Copy link
Contributor Author

@MikaelSmith
Thanks, I just have uploaded the patch on gerrit.
Walkthrough was really helpful.

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