Skip to content

Commit

Permalink
[KYUUBI #6905] PyHive HTTP/HTTPS dialect to use the database name fro…
Browse files Browse the repository at this point in the history
…m url

### Why are the changes needed?
HTTP dialect ignores the database specified in the URL and uses the "default" instead.

### How was this patch tested?
Tested manually.

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

Closes #6906 from developster/pyhive-update1.

Closes #6905

6e21d72 [Cheng Pan] Update python/pyhive/sqlalchemy_hive.py
ec7d462 [Octavian Ciubotaru] [KYUUBI #6905] PyHive HTTP/HTTPS dialect to use the database name from url

Lead-authored-by: Octavian Ciubotaru <[email protected]>
Co-authored-by: Cheng Pan <[email protected]>
Signed-off-by: Cheng Pan <[email protected]>
  • Loading branch information
developster and pan3793 committed Feb 14, 2025
1 parent 7c110b6 commit 2a2e4c2
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions python/pyhive/sqlalchemy_hive.py
Original file line number Diff line number Diff line change
Expand Up @@ -421,6 +421,7 @@ def create_connect_args(self, url):
"scheme": self.scheme,
"username": url.username or None,
"password": url.password or None,
"database": url.database or "default",
}
if url.query:
kwargs.update(url.query)
Expand Down

0 comments on commit 2a2e4c2

Please sign in to comment.