From 7b28bcef15cd37284780e605f5076ede2864afb4 Mon Sep 17 00:00:00 2001 From: Alex DeBrie Date: Tue, 21 Mar 2017 15:59:29 -0500 Subject: [PATCH] Fix documentation for adding a Redshift database (#2447) When adding a Redshift database, the docs said to use a connection string that started with `redshift+psycopg2://...`, but this results in a Python error from how SQLAlchemy executes the `get_schema_names` method (first reported [here](https://github.com/airbnb/superset/issues/2364#issuecomment-284705179)). The fix is to use `postgresql+psycog2://...` for the Redshift connection string. --- docs/installation.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/installation.rst b/docs/installation.rst index e85fd0176..714995088 100644 --- a/docs/installation.rst +++ b/docs/installation.rst @@ -196,7 +196,7 @@ Here's a list of some of the recommended packages. +---------------+-------------------------------------+-------------------------------------------------+ | sqlite | | ``sqlite://`` | +---------------+-------------------------------------+-------------------------------------------------+ -| Redshift | ``pip install sqlalchemy-redshift`` | ``redshift+psycopg2://`` | +| Redshift | ``pip install sqlalchemy-redshift`` | ``postgresql+psycopg2://`` | +---------------+-------------------------------------+-------------------------------------------------+ | MSSQL | ``pip install pymssql`` | ``mssql://`` | +---------------+-------------------------------------+-------------------------------------------------+