Connect: Fix and improve connection strings

crate / crate-clients-tools

Clients, tools, and integrations for CrateDB.

https://crate.io/docs/clients/

Apache License 2.0

3 stars 1 forks source link

Connect: Fix and improve connection strings #151

Closed amotl closed 2 days ago

amotl commented 1 week ago

About

A few fixes/improvements to different connection strings we are presenting on the documentation. For example:

Add missing ssl=true argument on SQLAlchemy connection string.
Remove /<dbname> argument from SQLAlchemy connection string, because it would cause errors.

Preview

https://crate-clients-tools--151.org.readthedocs.build/en/151/connect/

/cc @surister, @simonprickett

kneth commented 4 days ago

I have removed me as reviewer as my SQL Alchemy and Apache Flink knowledge is too shallow

amotl commented 2 days ago

I think this yak shaving went well? Let me know if you see any spots for improvements, or if you would do it completely different.

-- https://crate-clients-tools--151.org.readthedocs.build/en/151/connect/

Please check, also @karynzv, @hammerhead, @wierdvanderhaar, @zolbatar, @lservini, if you can spot something to be improved in this section of the documentation, also in general beyond this widget. Thank you.

NB: Dark mode courtesy of Furo and @msbt. 🚀