Skip to content

docs: databend-driver for python #32

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Nov 14, 2023
Merged

docs: databend-driver for python #32

merged 4 commits into from
Nov 14, 2023

Conversation

@soyeric128
Copy link
Contributor

@soyeric128 soyeric128 commented Nov 14, 2023

Introduced databend-driver.
@vercel
Copy link

vercel bot commented Nov 14, 2023 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
databend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 14, 2023 0:37am
@soyeric128
Copy link
Contributor Author

soyeric128 commented Nov 14, 2023 •
edited
Loading

I drafted the code with AsyncDatabendClient and it worked fine from my computer. It would be helpful if you help confirm these doubts:

  • i noticed the connection code containing the parameter sslmode, what else parameters available here?
  • Please give an example connecting to Databend Cloud. Thanks.
@soyeric128 soyeric128 requested a review from BohuTANG November 14, 2023 03:52
@everpcpc
Copy link
Member

everpcpc commented Nov 14, 2023 •
edited
Loading

The connection params and DSN are the same with BendSQL
@hantmac
Copy link
Member

hantmac commented Nov 14, 2023

I'm wondering whether we hidden databend-py directly or providing both databend-py and databend-driver(python binding). I think providing both options to the users would be better.
hantmac
hantmac approved these changes Nov 14, 2023
View reviewed changes
Copy link
Member

@hantmac hantmac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/LGTM
hantmac
hantmac reviewed Nov 14, 2023
View reviewed changes
@soyeric128 soyeric128 merged commit 5053593 into databendlabs:main Nov 14, 2023
@soyeric128 soyeric128 mentioned this pull request Nov 14, 2023
Closed
@soyeric128 soyeric128 deleted the py-driver branch November 14, 2023 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants

@soyeric128 @everpcpc @hantmac