You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by cr...@apache.org on 2020/05/20 20:13:46 UTC

[incubator-superset] branch master updated: fix: Emit a warning message rather than an exception on query failure (#9811)

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

craigrueda pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-superset.git


The following commit(s) were added to refs/heads/master by this push:
     new 13c2437  fix: Emit a warning message rather than an exception on query failure (#9811)
13c2437 is described below

commit 13c24372b026ddeda831c36970828f04a5e2897d
Author: Will Barrett <wi...@preset.io>
AuthorDate: Wed May 20 13:13:28 2020 -0700

    fix: Emit a warning message rather than an exception on query failure (#9811)
    
    * Emit a warning message rather than an exception on query failure
    
    * Add exc_info=True to warning message
---
 superset/connectors/sqla/models.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/superset/connectors/sqla/models.py b/superset/connectors/sqla/models.py
index 0182a9b..ad45474 100644
--- a/superset/connectors/sqla/models.py
+++ b/superset/connectors/sqla/models.py
@@ -1082,7 +1082,7 @@ class SqlaTable(Model, BaseDatasource):
         except Exception as ex:
             df = pd.DataFrame()
             status = utils.QueryStatus.FAILED
-            logger.exception(f"Query {sql} on schema {self.schema} failed")
+            logger.warning(f"Query {sql} on schema {self.schema} failed", exc_info=True)
             db_engine_spec = self.database.db_engine_spec
             errors = db_engine_spec.extract_errors(ex)