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

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

Commit 424c83bddcdefa4c0913f05bb72315ef1b5e2b7c in airflow's branch 
refs/heads/v1-10-test from Guy Maliar
[ https://gitbox.apache.org/repos/asf?p=airflow.git;h=424c83b ]

[AIRFLOW-5345] Allow SqlSensor's hook to be customized by subclasses (#5946)

Due to security reasons we have an alternative PostgreSQL hook that fetches 
credentials dynamically.
We would still like to use the `SqlSensor` functionality as is as our hook is 
still backed by sqlalchemy.
By extracting SqlSensor's `connection.get_hook` into a method we can override 
the `_get_hook` method without affecting the functionality of the hook.

(cherry picked from commit a8cb800662d674369d17d773fcfd3fcfac6bb2f5)


> Extract SqlSensor's get_hook functionality to an auxiliary function
> -------------------------------------------------------------------
>
>                 Key: AIRFLOW-5345
>                 URL: https://issues.apache.org/jira/browse/AIRFLOW-5345
>             Project: Apache Airflow
>          Issue Type: Improvement
>          Components: operators
>    Affects Versions: 1.10.4
>            Reporter: Guy
>            Assignee: Guy
>            Priority: Minor
>             Fix For: 1.10.7
>
>
> Due to security reasons we have an alternative PostgreSQL hook that fetches 
> credentials dynamically.
> We would still like to use the SqlSensor functionality as is as our hook is 
> still backed by sqlalchemy.
> By extracting SqlSensor's get_hook into a method we can override the 
> _get_hook method without affecting the functionality of the hook.
>  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to