-
Notifications
You must be signed in to change notification settings - Fork 13.6k
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
docs: improve documentation(docs): clarify URL encoding requirement for connection strings #30047
base: master
Are you sure you want to change the base?
Conversation
Adding extra information for the better understanding of connection string which is used for the backend of Superset. Based on this [Issue](apache#29979) People are finding difficulties in understanding the connection string, The `SQLAlchemy` documentation clearly talks about this URL encoded connection string, but users of superset not required to read / understand the concept of `SQLAlchemy` so adding this extra information may help them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the pull request! You wrote:
Is this addition specific to SQL Server only or should it be added to this page in general? |
Thanks for your appreciation, From the And one more information related to this is already the documentation for AWS Athena has information related to this but specific to Athena database. |
I had provided the wrong link earlier. I’ve now updated it with the correct issue discussion. I apologize for any confusion caused. |
SUMMARY
Enhance the documentation of the connection string used in the backend of Superset by adding additional information for improved understanding.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
TESTING INSTRUCTIONS
SQL Server
section onConnecting to Databases
page (/docs/configuration/databases#sql-server
).ADDITIONAL INFORMATION
See more discussions on this issue #29979