[
https://issues.apache.org/jira/browse/AIRFLOW-4587?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16872069#comment-16872069
]
ASF subversion and git services commented on AIRFLOW-4587:
----------------------------------------------------------
Commit 129d7dabc2fefff8c345015a9c65760e0959f0d1 in airflow's branch
refs/heads/v1-10-test from eladkal
[ https://gitbox.apache.org/repos/asf?p=airflow.git;h=129d7da ]
[AIRFLOW-4587] Replace self.conn with self.get_conn() in AWSAthenaHook (#5462)
(cherry picked from commit c8c7126593921322cc4f131ee318607cb893520c)
> AWS Athena Hook 'None' connection
> ---------------------------------
>
> Key: AIRFLOW-4587
> URL: https://issues.apache.org/jira/browse/AIRFLOW-4587
> Project: Apache Airflow
> Issue Type: Bug
> Components: aws, contrib, hooks
> Affects Versions: 1.10.3
> Reporter: João Ponte
> Priority: Trivial
> Fix For: 1.10.4
>
>
> Problem:
> In the following hook:
> [https://github.com/apache/airflow/blob/master/airflow/contrib/hooks/aws_athena_hook.py]
> There is a method get_conn() that initialises conn if it's none but every
> method in the hook uses conn directly, meaning it will be None when it gets
> used.
> Checking the AWS Athena Operator, you can see that get_conn() is called right
> after creating the hook to force initialisation. This is confusing and
> misleading.
>
> Solution:
> Replace all calls to self.conn with self.get_conn() to ensure initialisation.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
