[GitHub] [airflow] ephraimbuddy commented on pull request #9295: added mssql to oracle transfer operator

2020-08-03 Thread GitBox
ephraimbuddy commented on pull request #9295: URL: https://github.com/apache/airflow/pull/9295#issuecomment-668202527 > Hi @ephraimbuddy I did rebase, any hint on how to continue? Consider using odbc provider as advised on the mssql hook. The pymssql has been deprecated. I didn't tak

[GitHub] [airflow] ephraimbuddy commented on pull request #9295: added mssql to oracle transfer operator

2020-07-31 Thread GitBox
ephraimbuddy commented on pull request #9295: URL: https://github.com/apache/airflow/pull/9295#issuecomment-667353044 > @ephraimbuddy rebase master in my branch? See https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#how-to-rebase-pr

[GitHub] [airflow] ephraimbuddy commented on pull request #9295: added mssql to oracle transfer operator

2020-07-30 Thread GitBox
ephraimbuddy commented on pull request #9295: URL: https://github.com/apache/airflow/pull/9295#issuecomment-665875397 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [airflow] ephraimbuddy commented on pull request #9295: added mssql to oracle transfer operator

2020-06-18 Thread GitBox
ephraimbuddy commented on pull request #9295: URL: https://github.com/apache/airflow/pull/9295#issuecomment-646307048 Also @jkbngl you can read about it here https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#backport-providers-packages ---

[GitHub] [airflow] ephraimbuddy commented on pull request #9295: added mssql to oracle transfer operator

2020-06-17 Thread GitBox
ephraimbuddy commented on pull request #9295: URL: https://github.com/apache/airflow/pull/9295#issuecomment-645617767 Check other operators in provider package and see how they are named, just remove the transfer appended to it. I suggest to put the operator in `providers/microsoft/mssql/o

[GitHub] [airflow] ephraimbuddy commented on pull request #9295: added mssql to oracle transfer operator

2020-06-17 Thread GitBox
ephraimbuddy commented on pull request #9295: URL: https://github.com/apache/airflow/pull/9295#issuecomment-645605008 Hi @jkbngl. Sorry, I didn't notice the location of your operator before now. We no longer have operator in `contrib` package. Please check the `providers` package and

[GitHub] [airflow] ephraimbuddy commented on pull request #9295: added mssql to oracle transfer operator

2020-06-15 Thread GitBox
ephraimbuddy commented on pull request #9295: URL: https://github.com/apache/airflow/pull/9295#issuecomment-644420454 You may want to look at pre-commit hooks, so you can always avoid static check errors https://github.com/apache/airflow/blob/master/STATIC_CODE_CHECKS.rst#pre-commit