You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by hu...@apache.org on 2021/02/18 19:40:53 UTC

[superset] 01/01: add logs to db test connection and post failures

This is an automated email from the ASF dual-hosted git repository.

hugh pushed a commit to branch db-connection-logs
in repository https://gitbox.apache.org/repos/asf/superset.git

commit a6f9c782a52ea59d83389befeec84cb3b59e8ae9
Author: hughhhh <hu...@gmail.com>
AuthorDate: Thu Feb 18 14:33:41 2021 -0500

    add logs to db test connection and post failures
---
 superset/databases/api.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/superset/databases/api.py b/superset/databases/api.py
index ba665c1..bc4e71c 100644
--- a/superset/databases/api.py
+++ b/superset/databases/api.py
@@ -238,6 +238,7 @@ class DatabaseRestApi(BaseSupersetModelRestApi):
         except DatabaseInvalidError as ex:
             return self.response_422(message=ex.normalized_messages())
         except DatabaseConnectionFailedError as ex:
+            logger.exception("Database connection failed")
             return self.response_422(message=str(ex))
         except DatabaseCreateFailedError as ex:
             logger.error(
@@ -607,6 +608,7 @@ class DatabaseRestApi(BaseSupersetModelRestApi):
             TestConnectionDatabaseCommand(g.user, item).run()
             return self.response(200, message="OK")
         except DatabaseTestConnectionFailedError as ex:
+            logger.exception("Database test connection failed")
             return self.response_422(message=str(ex))
 
     @expose("/<int:pk>/related_objects/", methods=["GET"])