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

Insufficient Permissions Error when trying to access table #1555

Merged
merged 10 commits into from
Sep 27, 2024

Conversation

KuuCi
Copy link
Contributor

@KuuCi KuuCi commented Sep 26, 2024

To resolve this when permissions thrown by spark are not properly wrapped.

Instead of regex every variant of security perm errors, it's a lot easier to just rewarp the errors thrown by spark. Three scenerios were manually tested and passed:

User doesn't have access to catalog
User doesn't have access to schema (but has access for catalog)
User doesn't have SELECT for table (has access to catalog + schema)

This way, it's a lot easier to catch the permission spark errors.

@KuuCi KuuCi requested a review from a team as a code owner September 26, 2024 22:40
Copy link
Collaborator

@dakinggg dakinggg left a comment

Choose a reason for hiding this comment

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

Please write a more complete PR description, the ES ticket is internal

Copy link
Collaborator

@dakinggg dakinggg left a comment

Choose a reason for hiding this comment

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

New version is much nicer :) Although I think you still need to update the tests

@KuuCi KuuCi merged commit 107d246 into main Sep 27, 2024
9 checks passed
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