[jira] [Commented] (AIRFLOW-822) Close the connection before throwing exception in BaseHook

["ASF subversion and git services (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/AIRFLOW-822?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15848548#comment-15848548 ] 

ASF subversion and git services commented on AIRFLOW-822:
---------------------------------------------------------

Commit 4b6c389b34d0496c8dde86d9b6e5625a53edd93d in incubator-airflow's branch refs/heads/v1-8-test from [~Fokko]
[ https://git-wip-us.apache.org/repos/asf?p=incubator-airflow.git;h=4b6c389 ]

[AIRFLOW-822] Close db before exception

The basehook contains functionality to retrieve
connections from the
database. If a connection does not exist it will
throw an exception.
This exception will be thrown before the
connection to the database
is closed. Therefore the session to the db might
stay open and linger.

Closes #2038 from Fokko/airflow-822


> Close the connection before throwing exception in BaseHook
> ----------------------------------------------------------
>
>                 Key: AIRFLOW-822
>                 URL: https://issues.apache.org/jira/browse/AIRFLOW-822
>             Project: Apache Airflow
>          Issue Type: Bug
>          Components: hooks
>            Reporter: Fokko Driesprong
>             Fix For: 1.8.1
>
>
> Hi Guys,
> The basehook contains functionality to retrieve connections from the database. If a connection doesn't exists it will throw an exception. This exception will be thrown before the connection to the database is closed. Therefore the session to the db might stay open.
> Cheers, Fokko



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)