[ 
https://issues.apache.org/jira/browse/AIRFLOW-1734?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Bolke de Bruin resolved AIRFLOW-1734.
-------------------------------------
       Resolution: Fixed
    Fix Version/s: 1.9.0

Issue resolved by pull request #2703
[https://github.com/apache/incubator-airflow/pull/2703]

> Sqoop Operator contains logic errors & needs options to pass more sqoop 
> options
> -------------------------------------------------------------------------------
>
>                 Key: AIRFLOW-1734
>                 URL: https://issues.apache.org/jira/browse/AIRFLOW-1734
>             Project: Apache Airflow
>          Issue Type: Bug
>          Components: contrib
>            Reporter: Ace Haidrey
>            Assignee: Ace Haidrey
>              Labels: patch
>             Fix For: 1.9.0
>
>
> After taking a look at the Sqoop Operator I have found a number of errors 
> with it. I will describe them in depth on the PR I will make, but it includes 
> assign wrong parameters, not hiding the users raw password in the logs 
> (though a method to do that had been created in the hook, wrong if 
> conditions, and the inability to add other sqoop options that would make 
> sense to have the option to pass those.
> I have made some remarks in past commits about it too in the case I'm 
> misinterpreting but regardless there are definitely errors.
> https://github.com/apache/incubator-airflow/pull/2177#pullrequestreview-70046796
> Here is my PR with change by change explanation:
> https://github.com/apache/incubator-airflow/pull/2703/files#diff-8e77f042c2e060bbfd60828431a91e9bL131



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to