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

[FEATURE]: Crawl and Apply grants for SQL UDFs #808

Closed
1 task done
mwojtyczka opened this issue Jan 17, 2024 · 0 comments · Fixed by #812
Closed
1 task done

[FEATURE]: Crawl and Apply grants for SQL UDFs #808

mwojtyczka opened this issue Jan 17, 2024 · 0 comments · Fixed by #812
Assignees
Labels
enhancement New feature or request migrate/groups Corresponds to Migrate Groups Step of go/uc/upgrade

Comments

@mwojtyczka
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues

Problem statement

SQL UDFs grants are currently not crawled and applied during groups migration.

Proposed Solution

Add sql udf crawler and update existing grants module.

Additional Context

No response

@mwojtyczka mwojtyczka added enhancement New feature or request needs-triage labels Jan 17, 2024
@mwojtyczka mwojtyczka self-assigned this Jan 17, 2024
@mwojtyczka mwojtyczka added this to UCX Jan 17, 2024
@github-project-automation github-project-automation bot moved this to Triage in UCX Jan 17, 2024
@mwojtyczka mwojtyczka changed the title [FEATURE]: Crawl and Apply grants of SQL UDFs [FEATURE]: Crawl and Apply grants for SQL UDFs Jan 17, 2024
@nfx nfx added migrate/groups Corresponds to Migrate Groups Step of go/uc/upgrade and removed needs-triage labels Jan 17, 2024
@nfx nfx closed this as completed in #812 Jan 18, 2024
nfx added a commit that referenced this issue Jan 18, 2024
#812)

## Changes
Added support for crawling grants and applying Hive Metastore UDF ACLs.

Resolves #808 

### Functionality 

- [x] added new crawler for udfs
- [x] added handling of udfs to the existing TableAclSupport
- [x] added new column to inventory permissions table for udfs

### Tests

- [x] tested manually
- [x] added unit tests
- [x] added integration tests

---------

Co-authored-by: Serge Smertin <259697+nfx@users.noreply.github.com>
@github-project-automation github-project-automation bot moved this from Triage to Archive in UCX Jan 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request migrate/groups Corresponds to Migrate Groups Step of go/uc/upgrade
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants