superset-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From maximebeauche...@apache.org
Subject [incubator-superset] branch master updated: sql_lab: re-raise exception in get_sql_results (#3111)
Date Wed, 02 Aug 2017 05:38:54 GMT
This is an automated email from the ASF dual-hosted git repository.

maximebeauchemin 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 90592d3  sql_lab: re-raise exception in get_sql_results (#3111)
90592d3 is described below

commit 90592d3e3d4d2009ae59fcea55c08c932522e154
Author: Riccardo Magliocchetti <riccardo.magliocchetti@gmail.com>
AuthorDate: Wed Aug 2 07:38:52 2017 +0200

    sql_lab: re-raise exception in get_sql_results (#3111)
    
    As caller expect it to raise an exception instead of returning
    None.
    
    Refs #3075
---
 superset/sql_lab.py | 1 +
 1 file changed, 1 insertion(+)

diff --git a/superset/sql_lab.py b/superset/sql_lab.py
index 55130cd..1d0e892 100644
--- a/superset/sql_lab.py
+++ b/superset/sql_lab.py
@@ -98,6 +98,7 @@ def get_sql_results(
         query.status = QueryStatus.FAILED
         query.tmp_table_name = None
         sesh.commit()
+        raise
 
 
 def execute_sql(ctask, query_id, return_results=True, store_results=False):

-- 
To stop receiving notification emails like this one, please contact
['"commits@superset.apache.org" <commits@superset.apache.org>'].

Mime
View raw message