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

[spark]Support change default database #2628

Closed
wants to merge 2 commits into from

Conversation

hddong
Copy link

@hddong hddong commented Jan 3, 2024

Purpose

Linked issue: close #2627

Sometimes default can't be accessed, we need support change default database.

Tests

Add testChangeDefaultDbWithHive

API and Format

No

Documentation

No

@YannByron
Copy link
Contributor

When default database can't be accessed?

@hddong
Copy link
Author

hddong commented Jan 4, 2024

When default database can't be accessed?

Hive with ranger, we usually restrict tenant access to the default database. When default database can't be accessed, paimon throws permission exception.

@JingsongLi
Copy link
Contributor

Flink also has default database.

@YannByron
Copy link
Contributor

Flink also has default database.

You mean that paimon can support to specify an other database as a default one when session starts up? @JingsongLi

In the case that the default database is limited to access by range, users should change database when submit sql in hive/spark-sql. And spark thrift-server throws exception if no database is provided in beeline cmd. IMO, it's reasonable.

@JingsongLi JingsongLi closed this Jan 8, 2024
@JingsongLi JingsongLi reopened this Jan 8, 2024
@JingsongLi
Copy link
Contributor

re-open to trigger test.

@JingsongLi
Copy link
Contributor

@hddong You can show your exception. Describe your problem.

@YannByron
Copy link
Contributor

@hddong what's the behavior if the default dataset you provided is not exist.

@JingsongLi
Copy link
Contributor

close inactive pr.

@JingsongLi JingsongLi closed this Oct 16, 2024
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.

[Feature] Support change default database with spark
3 participants