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"])