[
https://issues.apache.org/jira/browse/AIRFLOW-1734?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Ace Haidrey updated AIRFLOW-1734:
---------------------------------
Description:
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
was:
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
> 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
>
> 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
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
