Skip to content

WIP: add config for defaulting to INVOKER for hive views#7358

Closed
Navgeet wants to merge 1 commit intotrinodb:masterfrom
Navgeet:hive-views-invoker-config
Closed

WIP: add config for defaulting to INVOKER for hive views#7358
Navgeet wants to merge 1 commit intotrinodb:masterfrom
Navgeet:hive-views-invoker-config

Conversation

@Navgeet
Copy link
Copy Markdown

@Navgeet Navgeet commented Mar 19, 2021

In the INVOKER security mode, tables referenced in the view are accessed using the permissions of the query user (the invoker of the view). A view created in this mode is simply a stored query.

This is often the desired behavior when migrating to trino from hive where this is the default. So adding a config property to set view security mode to INVOKER.

@cla-bot
Copy link
Copy Markdown

cla-bot bot commented Mar 19, 2021

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please submit the signed CLA to cla@trino.io. For more information, see https://github.com/trinodb/cla.

@bitsondatadev
Copy link
Copy Markdown
Member

bitsondatadev commented Oct 21, 2022

👋 @Navgeet - this PR is inactive and doesn't seem to be under development, and it might already be implemented by #12221. If you'd like to continue work on this at any point in the future, feel free to re-open.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants