ASF subversion and git services commented on AIRFLOW-109:

Commit 6f4696ba2ef18d74be8c18080b8ea7b9419608fb in incubator-airflow's branch 
refs/heads/master from [~artwr]
[ https://git-wip-us.apache.org/repos/asf?p=incubator-airflow.git;h=6f4696b ]

[AIRFLOW-109] Fix try catch handling in PrestoHook

This addresses the issue with executing the SQL statement outside of
the try block. In the case of a syntax error in the statement, the
underlying library raises a Databases error which was meant to be
handled (i.e., json parsed) by the catch.

> PrestoHook get_pandas_df executes a method that can raise outside of the try 
> catch statement.
> ---------------------------------------------------------------------------------------------
>                 Key: AIRFLOW-109
>                 URL: https://issues.apache.org/jira/browse/AIRFLOW-109
>             Project: Apache Airflow
>          Issue Type: Bug
>          Components: hooks
>    Affects Versions: Airflow 1.8, Airflow 1.7.1, Airflow 1.6.2
>            Reporter: Arthur Wiedmer
>            Assignee: Arthur Wiedmer
>            Priority: Minor
>              Labels: Presto
> This issue occurs when a malformed SQL statement is passed to the 
> get_pandas_df method of the presto hook. Pyhive raises a DatabaseError 
> outside of the try catch, leading in the wrong kind of error being raised.

This message was sent by Atlassian JIRA

Reply via email to