You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by "Ace Haidrey (JIRA)" <ji...@apache.org> on 2017/10/19 02:17:00 UTC

[jira] [Created] (AIRFLOW-1734) Sqoop Operator contains logic errors & needs options to pass more sqoop options

Ace Haidrey created AIRFLOW-1734:
------------------------------------

             Summary: 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


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 passign 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




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