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

[KYUUBI #5376] Change the GetSchemas and GetTables functions on kyuubi to use DDL to obtain information #6978

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

zxl-333
Copy link

@zxl-333 zxl-333 commented Mar 12, 2025

Change the GetSchemas and GetTables functions on kyuubi to use DDL to obtain information, thereby resolving the issue of DBeaver bypassing Ranger's permission restrictions.

Why are the changes needed?

The ranger plugin has been enabled to control permissions. When using DBeaver to connect to Kyuubi, users without permission for the database and tables can still see the database and tables on the left side.
After applying this modification, the access to the database tables can be controlled through the Ranger plugin. When users use DBeaver, they can only see the database tables for which they have permissions in the "Tables" column on the left side.

How was this patch tested?

Use the ranger plugin of Kyuubi for authorization, and use DBeaver to connect to Kyuubi for testing.

Was this patch authored or co-authored using generative AI tooling?

No

… kyuubi to use DDL to obtain information, thereby resolving the issue of DBeaver bypassing Ranger's permission restrictions.

The ranger plugin has been enabled to control permissions. When using DBeaver to connect to Kyuubi, users without permission for the database and tables can still see the database and tables on the left side.
After applying this modification, the access to the database tables can be controlled through the Ranger plugin. When users use DBeaver, they can only see the database tables for which they have permissions in the "Tables" column on the left side.

Use the ranger plugin of Kyuubi for authorization, and use DBeaver to connect to Kyuubi for testing.

No
@pan3793
Copy link
Member

pan3793 commented Mar 17, 2025

@zxl-333 the test failure is related, could you please fix them first?

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.

2 participants